halfshell icon indicating copy to clipboard operation
halfshell copied to clipboard
verified publisher icon rafikk

Image grayscale processor

Open mzsanford opened this issue 11 years ago • 2 comments

Noticed a note on the blog post suggesting it. There are several different ways to go about it but this seemed like the simplest and it required the least complex configuration.

mzsanford avatar Mar 14 '14 22:03 mzsanford

Thanks for working on this! This is a good start.

This works for setting up a route to grayscale images by default, but I think we'd also like to allow this via the URL api as well. So we should add this as a member on ImageProcessorOptions and parse it out from the query parameters in SourceAndProcessorOptionsForRequest as well.

I also wonder if there should be a way to enable/disable the feature in the configuration in addition to setting a default.

rafikk avatar Mar 16 '14 16:03 rafikk

@mzsanford The image processing API has undergone some refactoring in master. The statsd changes you made are already in master. If you refactor on top of master your PR should end up very small and easy to merge.

artnez avatar Nov 14 '14 00:11 artnez