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

Change deprecated ANTIALIAS to LANCZOS #66

Closed
wants to merge 2 commits into from

Conversation

ademaro
Copy link
Contributor

@ademaro ademaro commented Jul 16, 2023

Image.ANTIALIAS — no longer works in version 10
Image.LANCZOS — works since version 2.6, but is also deprecated
Image.Resampling.LANCZOS — modern version

#64

Reference: Pillow 10.0.0 release notes (with table of removed constants)

ANTIALIAS — not working 10+
LANCZOS — working since 2.6, but deprecated
Resampling.LANCZOS — modern version

matthewwithanm#64
@GHPS
Copy link

GHPS commented Jul 27, 2023

Works great fixing the issue Pilkit has with Pillow 10 -
please merge this PR...

@ademaro
Copy link
Contributor Author

ademaro commented Aug 2, 2023

Update: Image.LANCZOS deprecated in Pillow 9.1.0, but that was reversed in Pillow 9.4.0 and this constant will now remain available. Therefore, for maximum backward compatibility, I returned the constant name.

But this Pull Request is necessary since the ANTIALIAS constant has been completely removed in Pillow 10.
@matthewwithanm @vstoykov

@ademaro ademaro changed the title Change deprecated ANTIALIAS to Resampling.LANCZOS Change deprecated ANTIALIAS to LANCZOS Aug 2, 2023
@aljazkosir
Copy link

@matthewwithanm any update on this?

@matthewwithanm
Copy link
Owner

@matthewwithanm any update on this?

Sorry, @vstoykov is the one maintaining the project at the moment.

@xrmx xrmx mentioned this pull request Sep 26, 2023
@vstoykov
Copy link
Collaborator

Merged in #68

@vstoykov vstoykov closed this Sep 26, 2023
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.

5 participants