-
Notifications
You must be signed in to change notification settings - Fork 70
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
Region support #59
Comments
Hello. I finally managed to secure an hour or two to myself last night and took a look at it. Luckily the bug was easy to spot and fix. The problem turned out to be the format of XMP. Specifically: nested Anyway. It is fixed in the master right now and I'll try to release it sometime today or tomorrow. |
Thank you for your work! I'm using Adobe Lightroom to set these tags. I do not know if that is valid, but def. a well known app :) |
Hello @bpatrik. I'm happy to announce v7.0.0. It fixes a bug in XMP which caused the missing region data. |
Thank you very much. I did a quick test and the feature seems to work. Thanks again! We haven't decided which lib to use, but your one looks really promising at the first look: bpatrik/pigallery2#277 (comment) |
Hi,
I'm looking for a "one exif reader above all" for my home project https://github.com/bpatrik/pigallery2 (currently I'm using 4). Exifr looks really promising.
After a quick testing, I see that unfortunately it does not support regions:
(From: http://metapicz.com/)
It recoginses the Area, but does not attach the Area label (i.e.: the type of the area, e.g.: face and the name of the area eg.: Name of the person) to it:
This feature is needed to get multiple, tagged people (faces) from the photo.
Is there any plan adding this feature?
Thank you!
If helps, here is how I get this value using
exifreader
: https://github.com/bpatrik/pigallery2/blob/36ecbcbd3d5d3e659ddf20add456982dcb960d3d/src/backend/model/threading/MetadataLoader.ts#L221-L270I used this photo for testing: https://github.com/bpatrik/pigallery2/blob/master/demo/images/IMG_6253.jpg
The text was updated successfully, but these errors were encountered: