X Tutup
The Wayback Machine - https://web.archive.org/web/20201026224635/https://github.com/Intervention/image/pull/1035
Skip to content
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

Increase accuracy for gd/imagick alpha channel #1035

Draft
wants to merge 12 commits into
base: master
from

Conversation

@dontEatMe
Copy link

@dontEatMe dontEatMe commented Jul 30, 2020

2 decimal digits accuracy for alpha channel do not cover the entire space of 256(imagick)/128(gd) values (101 values possible only)
fix for that
tests need be changed too (accuracy has been increased)

dontEatMe added 6 commits Jul 30, 2020
increase accuracy on test
increase accuracy on test
Increase accuracy for imagick alpha channel
@dontEatMe
Copy link
Author

@dontEatMe dontEatMe commented Jul 30, 2020

tests needs to be changed too (accuracy has been increased)

dontEatMe added 4 commits Jul 30, 2020
@dontEatMe dontEatMe changed the title Increase accuracy for imagick alpha channel Increase accuracy for gd/imagick alpha channel Jul 30, 2020
@dontEatMe
Copy link
Author

@dontEatMe dontEatMe commented Jul 30, 2020

Now that fix ready for gd too

@dontEatMe
Copy link
Author

@dontEatMe dontEatMe commented Jul 30, 2020

...and add missed check for gd tests from imagick tests

@dontEatMe
Copy link
Author

@dontEatMe dontEatMe commented Jul 30, 2020

: /
Needs to check values on tests with new accuracy, not ready yet

@dontEatMe dontEatMe marked this pull request as draft Jul 31, 2020
@dontEatMe
Copy link
Author

@dontEatMe dontEatMe commented Sep 16, 2020

TODO add test to binary compare original image and result of 1:1 image convert

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.
X Tutup