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

Load: Ensure single byte strings (GPS LatitudeRef and LongitudeRef) can be decoded and re-encoded properly #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

joshanne
Copy link

@joshanne joshanne commented Sep 7, 2023

Hi @davetapley ,
From what I've read this is the official repo for piexif? See: https://pypi.org/project/piexif/

Reading the EXIF tags from an image appears to be corrupting the GPSLatitudeRef and GPSLongitudeRef fields in the EXIF data. Writing this data back to their fields is resulting in some applications unable to process the geotag in the images.

@joshanne joshanne changed the title Load: Ensure single byte strings (GPS LatitudeRef and LongitudeRef) c… Load: Ensure single byte strings (GPS LatitudeRef and LongitudeRef) can be decoded and re-encoded properly Sep 7, 2023
@davetapley
Copy link
Member

@joshanne yes! Thanks for the contribution.

I need to actually do a release since taking ownership of this project (see hMatoba#122), so this is a good motivation. I will merged and release ASAP 🙏🏻

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.

2 participants