Bug 460464 - "Scan for faces" context menu in Faces view does not trigger scan
Summary: "Scan for faces" context menu in Faces view does not trigger scan
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Faces-Workflow (show other bugs)
Version: 7.9.0
Platform: Appimage Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-10-15 09:20 UTC by Jens
Modified: 2022-10-16 15:33 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 7.9.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jens 2022-10-15 09:20:48 UTC
STEPS TO REPRODUCE
1. Open faces view
2. right-click on any face or several selected faces
3. select "scan for faces"

OBSERVED RESULT
In digikam-debug.appimage as of 2022-10-12, this finishes at once but doesn't do anything.

EXPECTED RESULT
In digikam.appimage as of 2022-10-02, this worked (and was translated).
Comment 1 Maik Qualmann 2022-10-15 10:10:23 UTC
Scanning for face detection in an image with face regions in People View was wrong, see Bug 459969. The function is now face recognition. The translation no longer fits due to changes in git/master and will no longer be corrected for digiKam-7.9.0.

Maik
Comment 2 Maik Qualmann 2022-10-15 11:00:53 UTC
I just tested the AppImage again, the entry "scan for faces" is no longer available in the context menu, it is now correctly called "Recognize Faces".

Maik
Comment 3 Jens 2022-10-15 17:26:12 UTC
OK, that makes sense, thanks. It doesn't fit my workflow but that's another issue.
Comment 4 Jens 2022-10-15 17:38:45 UTC
Actually, I just realized this was my own suggestion. *facepalm* Sorry about the confusion.
Bug 459969
Comment 5 Maik Qualmann 2022-10-16 05:50:44 UTC
Git commit 78008e9bf6dd2b79de800ec75f4fcaec0e4c4c3e by Maik Qualmann.
Committed on 16/10/2022 at 05:49.
Pushed by mqualmann into branch 'master'.

bring back scan for faces as scan-only in context menu
Related: bug 459969

M  +1    -0    core/app/views/preview/itempreviewview.cpp
M  +13   -8    core/app/views/stack/itemiconview_search.cpp
M  +5    -8    core/app/views/stack/itemiconview_views.cpp

https://invent.kde.org/graphics/digikam/commit/78008e9bf6dd2b79de800ec75f4fcaec0e4c4c3e
Comment 6 Jens 2022-10-16 15:33:01 UTC
Wait ... now we have both options, is this correct?
Re-recognize the currently selected face, and rescan the whole image for faces?
I didn't expect you to undo this change, re-recognizing makes more sense here than rescanning.
But having both options would of course be the best way to go.