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

[feature request] thumbnails with names #14

Open
herrxyz opened this issue Aug 27, 2022 · 5 comments
Open

[feature request] thumbnails with names #14

herrxyz opened this issue Aug 27, 2022 · 5 comments

Comments

@herrxyz
Copy link

herrxyz commented Aug 27, 2022

hey there,
i just found your plugin and like it.
As I want to export my marked faces from lightroom to train another KI (which will become better in face recognition than Lightroom hopefully), could you add the person name to thumbnails as exif-data? please add only the one in thumbnail, not just everything from big picture.
As the data is already in exiftool-log, it shouldn't be that complicated to add it to thumbnails too.

@colin0brass
Copy link
Owner

Thanks for the suggestion. I will have a think about how that could work and be implemented. Do you already have a thought about which exif field(s) would be appropriate?

@colin0brass
Copy link
Owner

Sorry for the delay. I finally had a look at this today and have pushed v1.12 (tag & release) which I hope might do what you are after.

I hadn't previously looked at the exif content of the thumbnail files, so was surprised to see there was exif in there at all, but as I think you were implying, it contained all of the exif from the original file.
I have added a step (using exiftool) to replace the exif region data with a single named region for the person in the thumbnail.

I didn't find a simple way to visualise the new region information in an independent tool, so I would be keen to hear back if this works for you or not.

After publishing I spotted a few other exif fields which (at least in my test images) contained people names but that I haven't yet constrained to single person for thumbnail export, but I will hold-off doing anything further for now:

  • PersonInImage
  • Subject
  • WeightedFlatSubject

@herrxyz
Copy link
Author

herrxyz commented Oct 23, 2022

thanks a lot :-)
I do not know an easy way to look at regions except colored markings/boundaries, but at least for me they are not necessary.
to be honest for my usecase I dont care in which ExifField the data in thumbnails is, as long as it's only the name of the specific person in a single field

@colin0brass
Copy link
Owner

I've been back in now and updated the remaining exif fields (PersonInImage, Subject, WeightedFlatSubject) in thumbnail export which used to contain original list of people names - so now they should only contain the person in the thumbnail export.

I found that WeightedFlatSubject could not be updated in the previous version of ExifTool that I was using (12.04), but having updated to later version (12.49) the field was now correctly updatable. Therefore I have updated the version of ExifTool bundled in this plugin.

I also happen to have added image caption writing, which I have been intending for a while, to help with exporting & sharing images complete with any photo description (caption) information that there might be for it, but this defaults to "off" so you can ignore it if you don't need it.

See latest tag & release v1.14

While testing on Windows I saw a couple of times when it failed with a "nil value for photoDimension" on the first run, but I haven't managed to pin that down yet. I haven't seen any such issues on Mac so far. My best guess is something about initialisation order resulting in a failed read of the exif meta-data, but I haven't proven anything specific yet. So far it has always worked after a retry (or few).

@colin0brass
Copy link
Owner

(I've captured the "nil value for photoDimension" intermittent problem as a separate issue #15)

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

No branches or pull requests

2 participants