<div dir="ltr"><div>Hi Maik,</div><div><br></div><div>You're right that sorting faces needs info about the number of Unconfirmed, Confirmed faces. To get that I was using the <a href="https://invent.kde.org/graphics/digikam/-/blob/master/core/libs/database/tags/facetagseditor.cpp#L67">FaceTagsEditor::unconfirmedFaceTagsIfaces</a> (with a slight modification that it returns Unconfirmed Faces and not Unconfirmed+Unknown faces). This method takes in an ItemInfo and returns the list of Unconfirmed faces in that image.</div><div><br></div><div>I haven't yet committed to the remote repo. because I wasn't sure about this implementation, however I'll do that now, so that you and other mentors may have a look at this.</div><div><br></div><div>Kartik<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jun 13, 2020 at 10:17 AM Maik Qualmann <<a href="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Kartik,<br>
<br>
To sort faces you need the information about confirmed and unconfirmed faces <br>
in the ItemInfo class. The ItemInfo class does not currently provide this <br>
information. The sorting of faces must not affect the other views. Are the <br>
latest sort changes already in the branch? I'll take a closer look at your <br>
branch at the weekend.<br>
<br>
Maik<br>
<br>
Am Freitag, 12. Juni 2020, 14:17:57 CEST schrieb Kartik Ramesh:<br>
> Hello all,<br>
> <br>
> I was working on the next issue in my Project which is to modify sorting of<br>
> views so that all Unconfirmed Faces for any tag appear collectively (either<br>
> before or after) the set of Confirmed Faces. This enhances visibility for<br>
> the user, and allows easy confirm/reject operations.<br>
> <br>
> Sorting implementations for views are present in ItemSortSettings, so I did<br>
> a little modifications, and I've been able to sort the view as shown here<br>
> <<a href="https://imgur.com/a/xODQfgj" rel="noreferrer" target="_blank">https://imgur.com/a/xODQfgj</a>>.<br>
> <br>
> However there's an issue with this, we're not sorting Faces but instead<br>
> ItemInfos. Since every view (be it Albums view or People view) works on the<br>
> same model, this new sorting order will enforce changes on the other views<br>
> as well. I don't think it's possible for face A to appear before face B but<br>
> image A (which contains face A) appears after image B (please correct me if<br>
> wrong).<br>
> <br>
> One way to fix this would be to implement a new Sorting role just for this<br>
> purpose. I also have some more fixes in mind but I would like to hear other<br>
> thoughts on this.<br>
> <br>
> Kartik<br>
<br>
<br>
<br>
<br>
</blockquote></div>