-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
imsave() should allow to store EXIF tags #2314
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
Comments
Can someone provide instructions for reproduction to help confirm in later MPL versions? |
This may be supported now via #7349 |
That only added PNG, which doesn't technically support EXIF. I would guess the target here is JPG; the interface would be the same as #7349 probably, with a difference in implementation of course. |
One can now just rely on Pillow's exif support (https://pillow.readthedocs.io/en/stable/releasenotes/6.0.0.html#png-exif-data) and pass kwargs to Pillow via the pil_kwargs parameter. |
When processing an image using
all EXIF tags are lost.
Optional support for EXIF would be very useful for photo processing. What about the following?:
The text was updated successfully, but these errors were encountered: