Bug 380336

Wanted to add that there are few things that could be improved in how digikam behaves after you added a face region.

1. If you click on the icon to drop down the list of names for some reason the list starts from the bottom (Unconfirmed) - suggestion: start from the most used or from the last used at least (ideally in future: start from the suggested tag for the face)
2. The size of the pop-up window is too small (3 lines only) - suggestion: expand it to 5-7 lines
3. First line is taken by "Tags", the second one "People" - suggestion: do not show "Tags" and "People" at all, just list the names (people)
4. Natural reaction to scroll the list up or down is to use the mouse scroll wheel. digikam instead scroll the whole image up and down (if zoomed in) - suggestion: lock the image while the people pop-up window is open and scroll the list up/down despite the location of the mouse cursor
5. After you scrolled the image up and down the people pop-up window gets blocked and does not show up again (face region is being shown at the time though). Reopening of the image helps so does switching to previous/next and go back - suggestion: fix that 
6. If you selected a name from the list and clicked on it you still need to click on "confirm" after. If you don't do that digikam does not remember the name you picked. Suggestion: Do not ask to confirm after user clicked on a name from the list
7. The list starts from "Unknown" as mentioned above. If you use the scroll bar to scroll up and then accidentally moved the mouse down it scrolls the list down. Moving the mouse up does not scroll the list up. Suggestion: either make the behavior work for scrolling both ways (up and down) or disable it at all.
8. Sometimes mouse wheel does scroll the list up and down however it starts scrolling the image after it reaches the end of the list. Suggestion: same as above - lock the image and scroll only within the pop-up people window while it is open

Another suggestion:
After user added a face region and when the field "Who is this?" shows up it would be great to be able to hover the mouse over the "who is this" field and use the wheel to scroll through the people list up and down and then digikam would remember the selection without needing to click on "confirm"

I mentioned below that "Tags" and "People" being shown. Clicking on "People" actually saves a face tag "People" which should not have happened.

After 3 weeks of work, i finally completed the compilation of AppImage using Qt
5.11.3 + QWebkit 5.212.

New 6.1.0 pre-release AppImage bundle can be found here (64 bits only for the
moment) :

https://files.kde.org/digikam/

Please check if this bugzilla entry still valid.

Thanks in advance

Gilles Caulier

7.0.0-beta1 is out with new Face Recognition algorithm based on Deep
Learning/Neural Network API from OpenCV

https://download.kde.org/unstable/digikam/

Please test and give us a feedback

Thanks in advance
Gilles Caulier

Git commit 05b9a534d89234b7571d5dade8067c490ab896cf by Gilles Caulier.
Committed on 26/12/2019 at 17:30.
Pushed by cgilles into branch 'master'.

Faces Workflow simplication and improvements: remove face scan dialog and embed Face scan settings in People left sidebar.
disable settings view while a face job is running.
Catch face detector tool complete/cancelled signal to re-enable settings view.
By this way, this will prevent to run 2 concurrent face jobs at the same time.
Related: bug 398376, bug 412168

M  +45   -31   core/app/views/sidebar/leftsidebarwidgets.cpp
M  +1    -0    core/app/views/sidebar/leftsidebarwidgets.h
M  +1    -1    core/utilities/facemanagement/CMakeLists.txt
R  +15   -45   core/utilities/facemanagement/widgets/facescanwidget.cpp [from: core/utilities/facemanagement/widgets/facescandialog.cpp - 089% similarity]
R  +9    -11   core/utilities/facemanagement/widgets/facescanwidget.h [from: core/utilities/facemanagement/widgets/facescandialog.h - 079% similarity]
R  +8    -11   core/utilities/facemanagement/widgets/facescanwidget_p.h [from: core/utilities/facemanagement/widgets/facescandialog_p.h - 090% similarity]
M  +1    -1    core/utilities/maintenance/facesdetector.cpp
M  +3    -3    core/utilities/maintenance/facesdetector.h
M  +1    -0    core/utilities/maintenance/maintenancetool.cpp
M  +11   -3    core/utilities/maintenance/maintenancetool.h

https://invent.kde.org/kde/digikam/commit/05b9a534d89234b7571d5dade8067c490ab896cf

I gave the 7.0.0 beta1 a quick spin Windows. I haven't noticed any improvements in the face tagging interface unfortunately. I will try to record a screencast tomorrow to describe the issue a little bit.

Git commit 3d56943b33495a1374e3d185844ffb80eb99cbdf by Maik Qualmann.
Committed on 27/12/2019 at 20:23.
Pushed by mqualmann into branch 'master'.

try with set combobox max view items to 10
Related: bug 413923

M  +1    -0    core/libs/tags/widgets/addtagscombobox.cpp

https://invent.kde.org/kde/digikam/commit/3d56943b33495a1374e3d185844ffb80eb99cbdf

Hello,
I tried to show what I mean in this video:
https://youtu.be/rITxoesL9pQ

digikam 7.0.0 latest daily build
When I click on the drop down button there is still a collapsed list of "People" shown. I think it would make sense to show an expanded list of names right away.
I would not want to see People, BalooTags, etc.
What do you think?

Just chiming in, there is an issue here that tagged faces can exist outside the "People" tree dir. Animals, for example. So hiding anything but "People" doesn't seem like a good idea at first. 

Also, if pictures with faces are imported, and that other person used Digikam in another language, instead of "People" it would be the equivalent in their language. So, for instance, you would see "People", and "Personnes", both containing faces.

I personally believe that People shouldn't be part of the general tag tree, but maybe a separate list, and it the word "People" shouldn't be a tag in itself, in order to to avoid what I just mentioned with the language. But I don't think that would be easy to achieve. 

Bug 392008 is related to this.