<div dir="ltr">Hey Tobias<div><br></div><div>Some comments about the code -</div><div><br></div><div>1. The code seems to be licensed under GPL. In order to make it into a framework, it will need to be re-licensed. This library seems like an ideal candidate for becoming a framework.</div><div><br></div><div>2. The copyright header seems to say "Part of the Digikam Project". You may want to change that.</div><div><br></div><div>3. There is an empty TODO file</div><div><br></div><div>4. The coding style uses seems a little unorthdox. Could you perhaps add a link to where one can know what style is being followed? Maybe this could go in the README file.</div><div><br></div><div>5. Identity ABI - The Identity class seems to be missing a d pointer.</div><div><br></div><div>6. From what I understand, facial recognition is not particularly cheap. However, the APIs appear to be blocking. What is the correct way to use them? Spawn a new thread? Run under a QRunnable?</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 25, 2014 at 12:14 PM, Tobias Leupold <span dir="ltr"><<a href="mailto:tobias.leupold@web.de" target="_blank">tobias.leupold@web.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi list!<br>
<br>
Here's (most of) a mail I recently posted on the kde-graphics-devel mailing<br>
list (cf. the whole discussion: <a href="http://mail.kde.org/pipermail/kde-graphics-devel/2014-September/000484.html" target="_blank">http://mail.kde.org/pipermail/kde-graphics-devel/2014-September/000484.html</a>):<br>
<br>
I'm from the KPhotoAlbum project. We recently introduced support for face<br>
detection and face recognition using libkface (which is currently hosted in<br>
extragear/libs, cf. <a href="https://projects.kde.org/projects/extragear/libs/libkface/" target="_blank">https://projects.kde.org/projects/extragear/libs/libkface/</a><br>
and <a href="http://api.kde.org/extragear-api/libs-apidocs/libkface/libkface/html/index.html" target="_blank">http://api.kde.org/extragear-api/libs-apidocs/libkface/libkface/html/index.html</a>). Likely, this will be a part of the<br>
next release. Actually, we are the first project apart from Digikam (who<br>
initiated and maintain libkface) to use libkface.<br>
<br>
This leads to a problem: at the moment, libkface is distributed with the<br>
Digikam sources. There's an own git repository for the lib's sources in the<br>
KDE repositories, but there's no own release.<br>
<br>
Some distributors (e. g. Gentoo) have an own package for libkface<br>
nevertheless, others don't (e. g. Debian). So, at the moment, we would have to<br>
include libkface in our sources, as Digikam does (which would lead to<br>
collisions if both KPA and Digikam would be installed and is perhaps not a<br>
good idea) or we would have to pull the full Digikam package as a dependency<br>
if the user wants libkface support.<br>
<br>
Both "solutions" don't seem to be nice.<br>
<br>
Thus, Gilles Caulier from Digikam suggested to move libkface into<br>
kdegraphics/libs, so that it will be included into the normal KDE release<br>
cycle and will be independent of Digikam (Digikam apparently can't manage a<br>
separate lib release itself, the original discussion can be found at<br>
<a href="http://mail.kde.org/pipermail/digikam-devel/2014-September/075737.html" target="_blank">http://mail.kde.org/pipermail/digikam-devel/2014-September/075737.html</a>).<br>
<br>
The discussion on kde-graphics-devel finished with the statement this had to<br>
be determined by kde-core-devel. So what do you think?<br>
<br>
Yours, Tobias<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><span style="color:rgb(192,192,192)">Vishesh Handa</span><br>
</div>