Bug 392017

Summary: Merging, renaming and removing face tags
Product: [Applications] digikam Reporter: MarcP <iwannaberich>
Component: Faces-WorkflowAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles, iwannaberich, metzpinguin
Priority: NOR    
Version: 5.9.0   
Target Milestone: ---   
Platform: Microsoft Windows   
OS: Microsoft Windows   
Latest Commit: Version Fixed In: 6.0.0
Sentry Crash Report:

Description MarcP 2018-03-18 20:17:42 UTC
I think there is some functionality missing regarding face tag management.

Is it possible to change the name of a face tag in a picture (or selection of pictures) and assign it to another existing person?

Is it possible to combine two persons and their tags?

If the name of a face tag (in the People list) is renamed in the right click -> properties section, are these changes written to the pictures' metadata?

If you remove a Person, are the face rectangles in the pictures deleted or emptied?
Comment 1 Maik Qualmann 2018-03-19 12:51:43 UTC
For merging tags we have a bug report (Bug 331864). If the name is changed from a tag, it will not be updated the image metadata yet. Again, we have a bug report (Bug 386224). However, this can be a time-consuming story. Either via the maintenance tool or select the tag in the left sidebar, select all the images and call "Write metadata to images" from the menu.

Maik
Comment 2 MarcP 2018-03-19 13:28:52 UTC
I posted the picture in another bug thread, but it's also relevant here:

https://imgur.com/a/vvPPV

If you look carefully can see there are 3 duplicated entries (well, 6 tags which should be just 3) that should be fused. But since digikam treats tags and face regions in a similar way, it's impossible to know which are which, so the results of renaming them are a bit unpredictable at the moment.

I could use another software to do that and then update Digikam's database, but I can't do that because of bug #380876, where digikam does not delete old unused tags from the database.

In picasa, for instance, there's always the option to select a few faces, right click on them and "Move to the person's album -> <select the person in a list>" and those faces will be correctly renamed and saved in the picture's metadata. This simple option (I don't know if it would be easy to implement) would solve this issue once for all.
Comment 3 Maik Qualmann 2018-04-14 16:14:46 UTC
Git commit cfa88b7f0f8fae3e2aa44f152a93123ae9328a4e by Maik Qualmann.
Committed on 14/04/2018 at 16:13.
Pushed by mqualmann into branch 'master'.

remove in the DB all old image informations when rereading metadata from file
when all 8 metadata write options are enabled
Related: bug 380876, bug 352711
FIXED-IN: 6.0.0

M  +4    -1    NEWS
M  +35   -12   core/libs/database/coredb/coredb.cpp

https://commits.kde.org/digikam/cfa88b7f0f8fae3e2aa44f152a93123ae9328a4e
Comment 4 MarcP 2018-04-15 12:07:29 UTC
Hi. Thanks for working on this issue.

However, I suppose that this bug report has been closed because now the metadata and the database are sync'd when a tag is deleted.

I accidentally reported several issues in this bug, some of them are still valid. Mainly the ability to merge two face tags. Should I open a new bug just for the merging issue?
Comment 5 Maik Qualmann 2018-04-15 13:14:41 UTC
Merging tags is now also possible. It does not matter if it's a face tag. Just Drag & Drop the double tag into the path of the other.

Maik
Comment 6 Maik Qualmann 2018-04-15 13:20:06 UTC
Of course, both tags must have the same name and not be in the same path, otherwise a tag must be renamed. If there are spelling errors in the tags.

Maik
Comment 7 MarcP 2018-04-15 20:57:21 UTC
Oh, great then, thanks!.

Btw, what would happen in the case two face tags are under the same hierarchy but with different names. e.g. "People/Michael Jackson" and "People/Michael Jakson"? Would it be possible to rename the latter to the first just by drag and drop?
Comment 8 Maik Qualmann 2018-04-25 21:05:46 UTC
Git commit 8d0254b2fc969a92e09028394f213cb751c7efb2 by Maik Qualmann.
Committed on 25/04/2018 at 21:04.
Pushed by mqualmann into branch 'master'.

add support to merge different names by drag & drop

M  +21   -1    core/app/dragdrop/tagdragdrop.cpp
M  +93   -62   core/libs/album/albummanager.cpp
M  +13   -0    core/libs/album/albummanager.h

https://commits.kde.org/digikam/8d0254b2fc969a92e09028394f213cb751c7efb2