[digikam] [Bug 374225] New: Add possibility to remove face identities by removing tag

[Simon]

https://bugs.kde.org/show_bug.cgi?id=374225

            Bug ID: 374225
           Summary: Add possibility to remove face identities by removing
                    tag
           Product: digikam
           Version: 5.4.0
          Platform: Other
                OS: Linux
            Status: UNCONFIRMED
          Severity: wishlist
          Priority: NOR
         Component: Faces-Management
          Assignee: digikam-devel at kde.org
          Reporter: freisim93 at gmail.com
  Target Milestone: ---

When deleting a face-tag in the people part of the left sidebar a new dialog
asks whether the connected face identity should be removed from the database. 

Use case:
If a person was face-tagged by mistake or is no longer of interest, removing
the face tags from images and deleting the tag itself does not remove this
person from the face database. Thus when face recognition is done again, there
will be new proposed face-tags for the delete person. The only way to get rid
of it is to remove all training data. With this patch only a specific identity
(and its training data) can be removed.

Detail about implementation:
The methods identityForTags and tagForIdentity of FaceUtils should probably be
static, shouldn't it?
They don't use any other class members. I had to create an instance of
FaceUtils just to call one of these methods. The only place one of these
methods is used in existing codes the same has been done. That seems
unnecessary and wasteful. Or is this a normal practice in c++?

-- 
You are receiving this mail because:
You are the assignee for the bug.