[digikam] [Bug 336253] Create a New tag, Unconfirmed Faces [patch]

[Mario Frank]

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

Mario Frank <mario.frank at uni-potsdam.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mario.frank at uni-potsdam.de

--- Comment #24 from Mario Frank <mario.frank at uni-potsdam.de> ---
Created attachment 103438
  --> https://bugs.kde.org/attachment.cgi?id=103438&action=edit
Patch for unconfirmed Person tag

I second the feature request, too.

But I think using filters is not so nice. The user would have to mark the faces
before recognition. And he would have to mark all images he imports.
Meaning: When we have a Unknown tag, why don't we also have an Unconfirmed tag
where all unconfirmed images are shown? The user could see transparently
which faces need to be confirmed and does not need to add markings which he has
to remove at the end again.

I took some time at the weekend to find some way to accomplish that.
In fact it was not even this complex.

I introduced the new special tag Unconfirmed which is automatically created if
not present. In detection phase, faces are marked as unknown as usual.
In recognition phase, the faces are marked with the recognised face tag, as
usual. But additionally, they are connected as autodetected person to the
Unconfirmed tag. In this property, I store the id of the person as which the
face was recognised and also the face region. 

With small adoptions in the image lister all recognised faces are now shown
both in the face tag which was recognised (person A) and the Unconfirmed tag
(with the suggested tag being person A).

Confirming the person can now be done in both tags. And the confirmed identity
is set as expected. Confirming the face also removes the face from unconfirmed
tag.
Rejecting the face also removes the face from both the recognised tag and also
from the Unconfirmed tag. Rescanning for faces (detection) also removes the
connection
to the Unconfirmed tag if unconfirmed results should be thrown away.

Can someone revise my patch?

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