Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Fix Pillow compatibility in example#23881
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
Instead of resampling down the stinkbug image down to 64x64 via pillow code (which is a bit distracting for the tutorial, especially if you start having version gating like here), perhaps simpler would be to just crop down to its central region? e.g. img[175:225, 225:275] seems reasonable. (Obviously the surrounding text needs adjusting too.) |
To be fully honest, I didn't read another line of the code except for the problematic one... Doing that, do we actually need to specify the resampling scheme? I mean, it is really a Pillow-function that we are using, so maybe it is not that important? (Default is bicubic.) (Btw, we also provide an, untested, thumbnail function, although that reads the file, possibly pops it up, and then saves it, as well as returning a Figure.) |
I didn't really pay attention to the context either. |
2312f5a
to45c569c
CompareI changed it to not specify resampling scheme (as that sort of makes sense as well and my previous approach was wrong). |
oscargus commentedSep 13, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
It seems like the 6.0.0 was released yesterday with a 6.0.0.post1 release two hours ago. Maybe it is just to wait for wheels to be published? |
oscargus commentedSep 13, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Windows and OSX support is currently blocked:pikepdf/pikepdf#390 Pin version (except for Linux)? |
Yes |
…881-on-v3.6.xBackport PR#23881 on branch v3.6.x (Fix Pillow compatibility in example)
PR Summary
Shouldfix#23855 (comment)
PR Checklist
Tests and Styling
pytest
passes).flake8-docstrings
and runflake8 --docstring-convention=all
).Documentation
doc/users/next_whats_new/
(follow instructions in README.rst there).doc/api/next_api_changes/
(follow instructions in README.rst there).