libkface

[Tobias Leupold]

Hi list :-)

Just that this is not only a conversation between me and Gilles Caulier from
Digikam ;-)

Quite a lot of work is going on at the moment to detach libkface from Digikam
and make it an independent library, released with KDE itself. You can read the
whole story on the Digikam-devel, Kde-graphics-devel and kde-core-devel
mailing lists.

Hopefully, libkface will already be included as a separate lib in the next KDE
release. Some changes had to be made so that it could be accepted. I just
pushed the necessary changes to the KPA codebase to reflect these changes (so
git master now depends on the current libkface git master. At the moment, we
don't have a libkface release with the "new" API yet).

Cheers, Tobias

[Robert Krawitz]

Post by Tobias Leupold
Hi list :-)
Just that this is not only a conversation between me and Gilles Caulier from Digikam ;-)
Quite a lot of work is going on at the moment to detach libkface from Digikam and make it an independent library, released with KDE itself. You can read the whole story on the Digikam-devel, Kde-graphics-devel and kde-core-devel mailing lists.
Hopefully, libkface will already be included as a separate lib in the next KDE release. Some changes had to be made so that it could be accepted. I just pushed the necessary changes to the KPA codebase to reflect these changes (so git master now depends on the current libkface git master. At the moment, we don't have a libkface release with the "new" API yet).
Cheers, Tobias
_______________________________________________
KPhotoAlbum mailing list
https://mail.kdab.com/mailman/listinfo/kphotoalbum

Currently getting the following build error (I have kface 4.3.0 installed):

[ 6%] Building CXX object CMakeFiles/kphotoalbum.dir/FaceManagement/Recognizer.cpp.o
/home/rlk/sandbox/kphotoalbum/FaceManagement/Recognizer.cpp: In member function ‘void FaceManagement::Recognizer::changeIdentityName(QString, QString, QString)’:
/home/rlk/sandbox/kphotoalbum/FaceManagement/Recognizer.cpp:112:61: error: expression cannot be used as a function
m_recognitionDatabase.setIdentityAttributes(identity.id(), attributes);
^
/home/rlk/sandbox/kphotoalbum/FaceManagement/Recognizer.cpp: In member function ‘void FaceManagement::Recognizer::updateCategoryName(QString, QString)’:
/home/rlk/sandbox/kphotoalbum/FaceManagement/Recognizer.cpp:205:80: error: expression cannot be used as a function
m_recognitionDatabase.setIdentityAttributes(allIdentities.at(i).id(), attributes);
^
/home/rlk/sandbox/kphotoalbum/FaceManagement/Recognizer.cpp: In member function ‘QPair<QString, QString> FaceManagement::Recognizer::parseIdentity(KFaceIface::Identity)’:
/home/rlk/sandbox/kphotoalbum/FaceManagement/Recognizer.cpp:243:37: error: ‘class KFaceIface::Identity’ has no member named ‘attributesMap’
QStringList tagParts = identity.attributesMap()[QString::fromLatin1("fullName")].split(
^
make[2]: *** [CMakeFiles/kphotoalbum.dir/FaceManagement/Recognizer.cpp.o] Error 1
make[1]: *** [CMakeFiles/kphotoalbum.dir/all] Error 2
make: *** [all] Error 2
--
Robert Krawitz <***@alum.mit.edu>

MIT VI-3 1987 - Congrats MIT Engineers 6 straight men's hoops tourney
Tall Clubs International -- http://www.tall.org/ or 1-888-IM-TALL-2
Member of the League for Programming Freedom -- http://ProgFree.org
Project lead for Gutenprint -- http://gimp-print.sourceforge.net

"Linux doesn't dictate how I work, I dictate how Linux works."
--Eric Crampton

[Tobias Leupold]

You need the latest git master libkface, not 4.3.0. The errors you get are due
to the very changes I wrote about.

After clearing your build directory and running cmake again, the 4.3.0 version
of libkface should be recognized as too old (lib version 3.3.0 against the
needed version 3.4.0) and the whole face management functionality should be
disabled anyway ...

[Tobias Leupold]

Could not find a package configuration file provided by "Kface" (requested
KfaceConfig.cmake
kface-config.cmake
Add the installation prefix of "Kface" to CMAKE_PREFIX_PATH or set
"Kface_DIR" to a directory containing one of the above files. If "Kface"
provides a separate development package or SDK, be sure it has been
installed.
-- checking for module 'libkface>=3.4.0'
-- found libkface, version 3.4.0

Okay, then I think this is a problem with the CMakeLists definition. If the
requested version is not found, the KFACE_FOUND variable should not be set,
causing all this not being built (and thus avoiding the errors).

I fear this out of my skills ... perhaps someone coding C++ longer than six
months can help here (Johannes?! ;-)

Anyways, we definitely have to fix this before the next release is done.

[Tobias Leupold]

Post by Tobias Leupold
Okay, then I think this is a problem with the CMakeLists definition.

That's right: my bad, I set the wrong version number. We need 3.5.0, not
3.4.0. would you try the latest git master code? Commit 5f25022 should fix
this issue.

Tobias

[Robert Krawitz]

Post by Tobias Leupold
Okay, then I think this is a problem with the CMakeLists definition.

That's right: my bad, I set the wrong version number. We need 3.5.0, not 3.4.0. would you try the latest git master code? Commit 5f25022 should fix this issue.

That worked.
--
Robert Krawitz <rlk-FrUbXkNCsVf2fBVCVOL8/***@public.gmane.org>

MIT VI-3 1987 - Congrats MIT Engineers 6 straight men's hoops tourney
Tall Clubs International -- http://www.tall.org/ or 1-888-IM-TALL-2
Member of the League for Programming Freedom -- http://ProgFree.org
Project lead for Gutenprint -- http://gimp-print.sourceforge.net

"Linux doesn't dictate how I work, I dictate how Linux works."
--Eric Crampton