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

from exiftool to exifread python lib and use location data #117

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jaakla
Copy link

@jaakla jaakla commented May 22, 2019

Should fix #115

@jaakla
Copy link
Author

jaakla commented May 22, 2019

#116 simple fix is included here, as separate commit.

@jaakla jaakla changed the title from exiftool to exifread python lib from exiftool to exifread python lib and use location data May 22, 2019
@andrewning
Copy link
Owner

I like the idea of only having python but (I may be wrong) exifread seems to not have the same level of capabilities as exiftool. From a quick test some of the file formats didn't work and some of the filters and recursive search options also seem to be lost.

edwardwbarber added a commit to edwardwbarber/sortphotos that referenced this pull request Aug 6, 2023
Based on jaakla's work in andrewning#117
but replaces dependence on exifread back to original exiftool implementation.

Fixes andrewning#116
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.

Dont require exiftool
2 participants