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

[Henrique Santos Fernandes]

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

--- Comment #33 from Henrique Santos Fernandes  ---
Thank you guys.

I'vent really used face detection because of this.

Actually i need to update my digikam installation.

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[Mario Frank]

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

--- Comment #32 from Mario Frank  ---
(In reply to caulier.gilles from comment #30)
> Mario,
> 
> Turn on "Setup/Views/Tree-Views/Show a count of items..." option.
> Unconfirmed tag still at zero.
> 
> Gilles

Hey Gilles,
fixed that one, too.

Will see what I can fix in the next days. For today I'm done ... headache.

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[Mario Frank]

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

--- Comment #31 from Mario Frank  ---
Git commit f11e34f7c376e92d8820c81f743a7fdeaaa380a1 by Mario Frank.
Committed on 16/01/2017 at 20:20.
Pushed by mfrank into branch 'master'.

This patch introduces the functionality to filter tags for assigning.
Only face tags are given if configured so. The setup option is located in misc
setup settings.

Also, the Unconfirmed face tag now shows the count of contained faces.
Related: bug 262168
FIXED-IN: 5.5.0

M  +3-1NEWS
M  +10   -0app/items/overlays/assignnameoverlay.cpp
M  +3-0libs/database/dbjobs/dbjob.cpp
M  +2-0libs/settings/applicationsettings.cpp
M  +3-0libs/settings/applicationsettings.h
M  +10   -0libs/settings/applicationsettings_miscs.cpp
M  +3-0libs/settings/applicationsettings_p.cpp
M  +3-0libs/settings/applicationsettings_p.h
M  +11   -0utilities/facemanagement/assignnamewidget.cpp
M  +8-2utilities/setup/setupmisc.cpp

https://commits.kde.org/digikam/f11e34f7c376e92d8820c81f743a7fdeaaa380a1

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[bugzilla_noreply]

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

--- Comment #30 from caulier.gil...@gmail.com ---
Mario,

Turn on "Setup/Views/Tree-Views/Show a count of items..." option. Unconfirmed
tag still at zero.

Gilles

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[Mario Frank]

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

Mario Frank  changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/dig
   ||ikam/7a7c29c0b8b36bd30a190a
   ||6cb4dc5f43960034a8
 Resolution|--- |FIXED
   Version Fixed In||5.5.0
 Status|CONFIRMED   |RESOLVED

--- Comment #29 from Mario Frank  ---
Git commit 7a7c29c0b8b36bd30a190a6cb4dc5f43960034a8 by Mario Frank.
Committed on 16/01/2017 at 18:24.
Pushed by mfrank into branch 'master'.

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.
FIXED-IN: 5.5.0

M  +3-1NEWS
M  +10   -0libs/database/coredb/coredbconstants.cpp
M  +2-0libs/database/coredb/coredbconstants.h
M  +21   -3libs/database/item/imagelister.cpp
M  +25   -0libs/database/tags/facetags.cpp
M  +3-0libs/database/tags/facetags.h
M  +15   -1libs/database/tags/facetagseditor.cpp
M  +12   -0libs/database/tags/facetagsiface.cpp
M  +5-0libs/database/tags/facetagsiface.h

https://commits.kde.org/digikam/7a7c29c0b8b36bd30a190a6cb4dc5f43960034a8

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[bugzilla_noreply]

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

--- Comment #28 from caulier.gil...@gmail.com ---
Mario,

The patch work as expected here. perfect.

Wolfgang, 

As the workflow have changed for face recognition step, this must be described
in handbook.

Gilles

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[bugzilla_noreply]

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

--- Comment #27 from caulier.gil...@gmail.com ---
I take a look in patch, and it's sound fine. I don't yet tested it. I will do
it this evening.

Gilles

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[Mario Frank]

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

--- Comment #26 from Mario Frank  ---
(In reply to Simon from comment #25)
> I don't have time to look at it in the moment, but one question: Would
> it also be (easily) possible to show unconfirmed faces of just one
> person? Maybe a switch to show unconfirmed only, confirmed only or both
> instead of one "container" with all the unconfirmed results.

I currently see no easy way. The only thing that *could* be quite easy
is grouping by face tag.

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[Simon]

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

--- Comment #25 from Simon  ---
I don't have time to look at it in the moment, but one question: Would
it also be (easily) possible to show unconfirmed faces of just one
person? Maybe a switch to show unconfirmed only, confirmed only or both
instead of one "container" with all the unconfirmed results.

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[aliteralmind]

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

aliteralmind  changed:

   What|Removed |Added

 CC||aliteralmind-bugskde@yahoo.
   ||com

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[Mario Frank]

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

Mario Frank  changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #24 from Mario Frank  ---
Created attachment 103438
  --> https://bugs.kde.org/attachment.cgi?id=103438=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 watching all bug changes.

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

[Mike-Sbg]

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

Mike-Sbg  changed:

   What|Removed |Added

 CC||ss2...@hotmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[via KDE Bugzilla]

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

jfle...@alum.rpi.edu changed:

   What|Removed |Added

 CC||jfle...@alum.rpi.edu

--- Comment #23 from jfle...@alum.rpi.edu ---
I'm willing to work on this issue given minimal direction from core developers.

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[via KDE Bugzilla]

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

caulier.gil...@gmail.com changed:

   What|Removed |Added

Version|4.10.0  |5.0.0

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[Evert Vorster via KDE Bugzilla]

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

Evert Vorster  changed:

   What|Removed |Added

 CC||evert.vors...@yandex.com

--- Comment #22 from Evert Vorster  ---
Just using the newest DigiKam 5.0.0 now, and this is still an issue. 
I have many faces detected in my thousands of photos. 

The method of finding photos without a tag does not work, as the photo gets a
tag when there is another already tagged face in it. 

Really, we need a tag called unconfirmed face, and grouped by who the software
thinks it is.

-- 
You are receiving this mail because:
You are watching all bug changes.

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

[via KDE Bugzilla]

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

stwag...@openst.de changed:

   What|Removed |Added

 CC||stwag...@openst.de

-- 
You are receiving this mail because:
You are watching all bug changes.