Bug 342253 - How to enable face recognition is hard to detect in settings dialogue
Summary: How to enable face recognition is hard to detect in settings dialogue
Status: RESOLVED FIXED
Alias: None
Product: kphotoalbum
Classification: Applications
Component: Annotation Dialog (show other bugs)
Version: GIT master
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Johannes Zarl-Zierl
URL:
Keywords:
Depends on:
Blocks: 342457
  Show dependency treegraph
 
Reported: 2014-12-27 16:14 UTC by Andreas Schleth
Modified: 2016-01-17 09:41 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Schleth 2014-12-27 16:14:47 UTC
KPA git master (27.12.2014) compiled with KFace.  
The button for face detection is there, but inactive.  
How to enable face recognition in Kphotoalbum??
Solution: 
Enter Kphotoalbum settings -> Categories and select your "Person"-category.
For this category tick "positionable tags [x] coordinates can be added to this category".

Suggestion for improvement:
Maybe a label like 
"enable face recognition for this category [ ]" or
"link this category to face recognition [ ]" or similar
would be easier to understand.
Possibly it would be even better, if the category that is linked to face recognition would be selectable from a list in the settings dialogue for face recognition.

It is probably impossible for somebody outside the coders community to enable this great feature.  I had to look through the sources to find the solution.  Why?  Because I associated geo-coordinates with this label and not pixel-coordinates inside an image.
Comment 1 Johannes Zarl-Zierl 2014-12-30 21:51:17 UTC
For new databases, positionable tags are enabled for the category "People" by default, but I agree that the wording could be better.

"enable face recognition for this category" or something similar is not quite right, since positionable tags can be used without face recognition.
Comment 2 Johannes Zarl-Zierl 2014-12-31 16:29:55 UTC
Git commit b38577dc758a5b47722fe3aac9b95dce443fcefb by Johannes Zarl.
Committed on 31/12/2014 at 16:29.
Pushed by johanneszarl into branch 'master'.

Better wording in settings-text on positionable tags.

M  +1    -1    Settings/CategoryPage.cpp

http://commits.kde.org/kphotoalbum/b38577dc758a5b47722fe3aac9b95dce443fcefb
Comment 3 Andreas Schleth 2015-01-01 13:58:12 UTC
I like the new text much better.
Comment 4 Tobias Leupold 2015-01-01 13:59:39 UTC
Probably, we should leave the button enabled, even if there's no "positionable" category and simply display a warning like "To be able to use face detection, please enable 'Tags in this category can be associated with an area of the image' for at least one category in the settings dialog".

The description of the option is now better, but perhaps, the relation between a category that "can be associated with an area of the image" and face detection is still not obvious?

@Andreas: Thanks for filing this bug! It's true that if you implement stuff yourself, you don't see such pitfalls ;-)
Comment 5 Johannes Zarl-Zierl 2015-01-01 15:42:39 UTC
Git commit da31ab41c02036a4e1e59d03f137685d4f063602 by Johannes Zarl.
Committed on 01/01/2015 at 15:41.
Pushed by johanneszarl into branch 'master'.

Add hints on how to enable positionable tags if disabled.

M  +39   -10   AnnotationDialog/ImagePreviewWidget.cpp
M  +4    -0    AnnotationDialog/ImagePreviewWidget.h

http://commits.kde.org/kphotoalbum/da31ab41c02036a4e1e59d03f137685d4f063602
Comment 6 Tobias Leupold 2015-06-04 12:27:19 UTC
I think it's more clear now … so can we close this one?
Comment 7 Tobias Leupold 2016-01-17 09:41:44 UTC
Well, nobody ever complained again about this, so I'll close it now ;-)