-
Notifications
You must be signed in to change notification settings - Fork 141
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
Some png files when resized with png.quality results as empty/transparent png file #613
Comments
So, I created an integration test on the latest version - but it works on my machine. Can you verify this is the correct input file? Maybe GitHub reencoded it? Using w=1230&h=760&png.quality=75&mode=crop&scale=both, I get https://imageflow-resources.s3.us-west-2.amazonaws.com/visual_test_checksums/0E5E63543D35A149D.png |
Hmm, this is failing on windows but not linux. |
And yet even on windows the result looks correct, despite being compressed differently. Can the problem image be emailed to lilith@imazen.io to make sure GitHub isn't 'repairing' it? Also, can you try with the very latest Imageflow release? This was listed as being in an older version. |
So, GitHub apparently re-compresses images. The image you e-mailed me is a 24-bit PNG, 18.4KB, and the image GitHub lets me access is a 32-bit PNG, 27.5KB |
This was specific to PNG files with a zero alpha channel, encoded when using lossy PNG, and when no transparency-inducing operations are performed. Thank you for e-mailing the problematic file, that allowed me to diagnose the issue. |
We need to resize this
demo.png
image.Issue happens with parameters:
demo.png?w=1230&h=760&png.quality=75&mode=crop&scale=both
If we append
png.lossless=true
than resized image looks OK,except that we can not control quality
demo.png?w=1230&h=760&png.quality=75&mode=crop&scale=both&png.lossless=true
Issue happens with code that is using following dependencies:
Same code works as expected with older version of dependencies:
Same issue is tracked here: 2sic/2sxc#2932
Please check 2sic/2sxc#2932 (comment)
The text was updated successfully, but these errors were encountered: