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

PIL -> Pillow #2137

Merged
merged 2 commits into from Jan 22, 2014
Merged

PIL -> Pillow #2137

merged 2 commits into from Jan 22, 2014

Conversation

mdboom
Copy link
Member

@mdboom mdboom commented Jan 17, 2014

I'm opening an issue to initiate discussion centred around migrating from PIL to Pillow. A comment in PR #1965 inspired me to create this issue.

Commence open debate.

@tacaswell
Copy link
Member

👍 on doing this.

@efiring
Copy link
Member

efiring commented Jan 17, 2014

Right, it seems like the obvious thing to do ASAP.

@pelson
Copy link
Member

pelson commented Jan 17, 2014

I think we've already done this. https://github.com/matplotlib/matplotlib/blob/master/.travis.yml#L20

The only problem remaining is communicating the version that we support (and whether we will continue to support PIL going forwards).

@mdboom
Copy link
Member

mdboom commented Jan 17, 2014

I've attached some documentation improvements for this. We've actually supported Pillow for some time. It doesn't require much, as the APIs are for the most part the same.

@@ -1,8 +1,8 @@
#!/usr/bin/env python
"""
Use backend agg to access the figure canvas as an RGB string and then
convert it to an array and pass the string it to PIL for
rendering
convert it to an array and pass the string it to Pillow for
Copy link
Member

Choose a reason for hiding this comment

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

Maybe "pass the string it to Pillow" -> "pass it to Pillow".

tacaswell added a commit that referenced this pull request Jan 22, 2014
@tacaswell tacaswell merged commit db49c9c into matplotlib:master Jan 22, 2014
@mdboom mdboom deleted the doc/pil-version branch August 7, 2014 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants