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

Mario Frank bugzilla_noreply at kde.org
Mon Jan 16 14:12:00 GMT 2017


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
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
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.

More information about the Digikam-devel mailing list