-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Change default resize resampling filter from NEAREST to BICUBIC #4255
Change default resize resampling filter from NEAREST to BICUBIC #4255
Conversation
@python-pillow/pillow-team |
Makes sense to me. |
@homm Go for it, pending coordination with @hugovk and @radarhere and maybe some extra "we're doing this" type announcements preceding the release (presumably Jan 1), particularly if it could break existing usage. |
Actually, there are Another question is |
Normally we would want to put deprecation warnings where possible, but that would not have had much effect here. The next release is a major, breaking release, so improving the default is fine. We're including it in the release notes, which I think is enough here. Should we mention in the release notes something along the lines that performance is slower but with improved downscaling and upscaling quality? |
@hugovk Maybe the link to filters description would be enough? https://pillow.readthedocs.io/en/stable/handbook/concepts.html#concept-filters |
… and ImageOps.fit()
Yep, looks good. |
skip-checks: true
@radarhere done |
I've created #5411 to revert this for special image modes, since they don't currently support any resampling other than NEAREST. |
I believe this is much convenient in most cases, which makes Pillow much user-friendly.