-
Notifications
You must be signed in to change notification settings - Fork 268
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
Bump Pillow to be sure it is a secure version #405
Conversation
.travis.yml
Outdated
fi | ||
- chmod +x miniconda.sh | ||
- ./miniconda.sh -b -p $HOME/miniconda | ||
- export PATH=$HOME/miniconda/bin:$PATH | ||
- conda update -q --yes conda | ||
# - conda update -q --yes conda |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this line? I don't see a reason to have it commented.
Some tests fail with the new pillow |
It looks like the old pillow had a bug there. I tested on images of bigger sizes and pillow 3.4 -- 5.2 works fine and the same here. Not sure why old pillow had this answer, but in this test the rotation angle is 12.25 degrees. It means that the difference in the output image will be less than 1 pixel. So, the new pillow is giving the right answer. |
Basically, it fails now only because of Feel free to merge both PRs. |
ea11404
to
7219ffe
Compare
Tests passed, I merge |
No description provided.