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

Added IPTC parsing by pyExiv2 module #116

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

Conversation

yozkiyokly
Copy link

For photographers who taging theis collections using IPTC metadata. IPTC is much flexible than EXIF because IPTC supports more descriptive text fields (EXIF have only one - "comment" field).
In this release IPTC "title" field have a little priority over EXIF "comment: field.
Part of this logic implemented inside exifread code block.
Added module pyExiv2 linkage with several checks.
If user have no pyexiv2 installed, nothing happens.

I leave several huge comments blocks inside code.
This is my first python release, so excuse me please for unclean code.
It works, it parses. Three days of work and it is here.

For photographers who taging theis collections using IPTC metadata. 
It is much flexible than EXIF because IPTC supports more descriptive text fields (EXIF have only one - "comment"  field).
In this release IPTC "title" field have a little priority over EXIF "comment: field. Part of this logic implemented inside exifread code block.
Added module pyExiv2 linkage with several checks.  If user have no pyexiv2 installed, nothing happens.
I leave several huge comments inside code. 
This is my first python release, so excuse me please for unclean code.
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.

None yet

1 participant