Bug 342253

Summary:	How to enable face recognition is hard to detect in settings dialogue
Product:	[Applications] kphotoalbum	Reporter:	Andreas Schleth <schleth_es>
Component:	Annotation Dialog	Assignee:	Johannes Zarl-Zierl <johannes>
Status:	RESOLVED FIXED
Severity:	normal	CC:	johannes, tl
Priority:	NOR
Version:	GIT master
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Bug Depends on:
Bug Blocks:	342457

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 ;-)