Skip to content

Remove pin now Pillow issue is fixed#17043

Merged
rth merged 1 commit intoscikit-learn:masterfrom
hugovk:patch-1
Apr 26, 2020
Merged

Remove pin now Pillow issue is fixed#17043
rth merged 1 commit intoscikit-learn:masterfrom
hugovk:patch-1

Conversation

@hugovk
Copy link
Copy Markdown
Contributor

@hugovk hugovk commented Apr 26, 2020

Reference Issues/PRs

Fixes #16824.

What does this implement/fix? Explain your changes.

The pin was temporarily added in #16843.

python-pillow/Pillow#4518 has been fixed and was released yeterday in Pillow 7.1.2.

https://pillow.readthedocs.io/en/stable/releasenotes/7.1.2.html

Any other comments?

Thanks for maintaining scikit-learn!

Copy link
Copy Markdown
Member

@rth rth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @hugovk ! Also thank you for maintaining pillow!

@rth rth merged commit e8a42ae into scikit-learn:master Apr 26, 2020
@hugovk hugovk deleted the patch-1 branch April 26, 2020 11:54
@hugovk
Copy link
Copy Markdown
Contributor Author

hugovk commented Apr 26, 2020

You're welcome!

@nikhildhurve
Copy link
Copy Markdown

Thanks

gio8tisu pushed a commit to gio8tisu/scikit-learn that referenced this pull request May 15, 2020
viclafargue pushed a commit to viclafargue/scikit-learn that referenced this pull request Jun 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI failure with doctest on unsupervised_learning.rst with pillow 7.1.0

4 participants