-
Notifications
You must be signed in to change notification settings - Fork 857
Add config var to specify desired png compression #751
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@kkopachev we should add ssim tests for this optimization. |
|
Not sure we need that. Apart from that, looking at the pillow source, I think there is optimize option available for gif as well ❔ |
|
New plan: instead of forcing max compression, expose config var. This is useful if people use more efficient optimizers after image operations. In case of PNG there is no reason to compress it, decompress and then compress it harder one more time. |
|
Any updates here? |
|
Guys, should I change something? Or explain a little more how that's useful? |
|
Thanks @kkopachev . We were expecting a difference in output but as you stated this compression happens at the file itself and is not lossy. |
I noticed this option is set for jpegs, so why not for png?
Or the idea here to let people use optimizers after main transformations?