Bug 415796 - face thumbnail zoom level too wide
Summary: face thumbnail zoom level too wide
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Faces-Workflow (other bugs)
Version First Reported In: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-01-02 12:21 UTC by Daniel
Modified: 2020-01-02 20:27 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 7.0.0
Sentry Crash Report:


Attachments
the thumbnail in the unconfirmed/unknown faces view (41.12 KB, image/png)
2020-01-02 12:21 UTC, Daniel
Details
the actual face rectangle when click on the image (71.04 KB, image/png)
2020-01-02 12:22 UTC, Daniel
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel 2020-01-02 12:21:19 UTC
Created attachment 124850 [details]
the thumbnail in the unconfirmed/unknown faces view

SUMMARY


With the new detection, the face thumbnail shows a bit more than the face rectangle, which is confusing if there are multiple people close to each other or a person is in one corner of the face thumbnail.


Using version with latest commit 6e2c4603dbebcd09a110d214c08aeb6dbdd8b16a
Comment 1 Daniel 2020-01-02 12:22:12 UTC
Created attachment 124851 [details]
the actual face rectangle when click on the image
Comment 2 Maik Qualmann 2020-01-02 20:23:40 UTC
We always added a fixed value of 70 pixels in all directions for the thumbnail. This is fine for big images with big faces. With small faces, too much environment is shown. I have now changed this and we are using 10% of the longest side of the face rectangle as the value. Although this will now add new thumbnails to the database, it might be a good idea to start with digiKam-7.0.0 with an empty thumbnail DB.

Maik
Comment 3 Maik Qualmann 2020-01-02 20:27:44 UTC
Git commit 1fa3e45ac80083b5b738b9a912d068adb19e4dd6 by Maik Qualmann.
Committed on 02/01/2020 at 20:26.
Pushed by mqualmann into branch 'master'.

change face thumbnail zoom level
FIXED-IN: 7.0.0

M  +1    -1    NEWS
M  +1    -1    core/app/items/delegate/itemfacedelegate.cpp
M  +6    -3    core/utilities/facemanagement/database/faceutils.cpp
M  +1    -1    core/utilities/facemanagement/database/faceutils.h

https://invent.kde.org/kde/digikam/commit/1fa3e45ac80083b5b738b9a912d068adb19e4dd6