<div dir="ltr"><div dir="ltr"><div>Thanks for clarifying, Gilles.</div><div>Yes, as Trung mentioned, the purpose of the red round button is not to Ignore Faces. Trung, your observations are correct except for :</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div> Clicking on the "<b>-</b>" button turns an unconfirmed face to ignored face</div></blockquote><div> Ignored -> Unknown. As we are rejecting the face suggestion.<br></div><div><strike><br></strike></div><div>Currently the functionality of the buttons is as follows:</div><div>1. Red Round Button (top right) : delete Face from database, regardless of what type of face it is. The red round button currently shows the tool-tip "If this is not a face, click to reject it", maybe I could modify it to emphasize that the face will be deleted.</div><div>2. (-) Button (bottom right) : move face to Unknown, except if face is already Unknown. In that case, move it to Ignored. This button is disabled for Ignored.<br></div><div>3. Green checkbox (bottom left) : Confirms the Face. For Unconfirmed Faces (i.e. suggestions), this corresponds to marking as Confirmed. For Ignored Faces, however, this leads to a marking of the face as Unknown. I preferred this because it seemed intuitive, as if the user is saying "Ok, I'm allowing this face to be detected now". I've displayed the tool-tip accordingly.<br></div><div><br></div><div>Also another thing, earlier Maik had mentioned that the Ignored Face shouldn't be storing the tag region in the database, as the user doesn't care for the region anymore. I liked the idea at first, and implemented the tagRegion for Ignored tag as a NULL string but I ran into two issues :</div><div>1. How to detect overlaps of Face Region between faces being recognized (during a recognition process) and the faces marked as Ignored.</div><div>2. How to show the face in the Ignored View (when we selected Ignored tag from People Sidebar), as the default implementation to depict faces without a region is to show the entire image.</div><div><br></div><div>I could have worked around with a dummy placeholder (as Maik suggested), but I thought to keep the implementation simple. Anyhow, in the current implementation, the image is not being associated with the Ignored face, i.e. if you mark a face as Ignored, and then open that image (in Digikam album view), then the ignored face will not be displayed. So it works fine.</div><div><br></div><div>Kartik<br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, May 17, 2020 at 5:56 PM Gilles Caulier <<a href="mailto:caulier.gilles@gmail.com">caulier.gilles@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">This one probably :<br>
<br>
<a href="https://imgur.com/m5lftgN" rel="noreferrer" target="_blank">https://imgur.com/m5lftgN</a><br>
<br>
Gilles Caulier<br>
<br>
Le dim. 17 mai 2020 à 14:17, Kartik Ramesh <<a href="mailto:kartikx2000@gmail.com" target="_blank">kartikx2000@gmail.com</a>> a écrit :<br>
><br>
> Thank you for your responses :)<br>
> I will share my thought on the implementation, but before that could someone please clarify what is the "round red button" is being referred to? Maybe I'm using a different theme, so please clarify, because I don't want to start explaining without completely understanding the doubt.<br>
><br>
> Thanks<br>
> Kartik<br>
><br>
> On Sun, May 17, 2020 at 4:17 PM Maik Qualmann <<a href="mailto:metzpinguin@gmail.com" target="_blank">metzpinguin@gmail.com</a>> wrote:<br>
>><br>
>> Hmm, in my opinion the round red button should ignore the face and no longer<br>
>> display it in future face detections. The "-" button should make an<br>
>> unconfirmed face an unknown face again.<br>
>><br>
>> Maik<br>
>><br>
>> Am Sonntag, 17. Mai 2020, 12:07:24 CEST schrieb Thanh Trung Dinh:<br>
>> > The implementation is really good, well done Kartik!<br>
>> > Besides, talking about the red button, I think its functionality is not for<br>
>> > turning unknown faces to ignored faces. Clicking on the "*-*" button turns<br>
>> > an unconfirmed face to ignored face, but clicking red button will remove<br>
>> > face tag completely I suppose. Then, when you run face detection again with<br>
>> > option "skip already scanned images", those faces (rejected by the red<br>
>> > button) will not appear, but when you run again with option "scan again and<br>
>> > merge results", those faces will be rescanned and appear again if there are<br>
>> > faces detected. For me, the functionality of the red button is quite<br>
>> > confused. Correct me if my observation above is wrong, but I think that the<br>
>> > red button should be hidden on the overlay. What do you think?<br>
>> ><br>
>> ><br>
>> ><br>
>> > On Sun, May 17, 2020 at 11:07 AM Maik Qualmann <<a href="mailto:metzpinguin@gmail.com" target="_blank">metzpinguin@gmail.com</a>><br>
>> ><br>
>> > wrote:<br>
>> > > I like the implementation very much, good job. But if I ignore a face (the<br>
>> > > red<br>
>> > > round overlay button), it is found again. I did the manual detection on an<br>
>> > > image. When I look to the "ImageTagProperty" table the region was deleted<br>
>> > > and<br>
>> > > didn't get the "Ignored" Tag ID. So it will be found again.<br>
>> > ><br>
>> > > Maik<br>
>> > ><br>
>> > > Am Sonntag, 17. Mai 2020, 08:59:07 CEST schrieb Kartik Ramesh:<br>
>> > > > Hello all, I've managed to implement the feature for Ignored Faces. I've<br>
>> > > > pushed to the digikam repo. please let me know if it's appropriate.<br>
>> > > ><br>
>> > > > The basic idea is as follows :<br>
>> > > > 1. Pressing Reject on Unknown Faces, should mark the face as Ignored.<br>
>> > ><br>
>> > > This<br>
>> > ><br>
>> > > > will create a new tag, and move the face to it.<br>
>> > > > 2. Running a Face Detection now, will not re-detect the Ignored Face<br>
>> > ><br>
>> > > (even<br>
>> > ><br>
>> > > > if Scan again and Merge is selected). Facial Recognition will not try to<br>
>> > > > recognize the ignored faces, and will never suggest names for an ignored<br>
>> > > > face.<br>
>> > > > 3. I have re-designed overlay for Ignored Faces in particular. Pressing<br>
>> > ><br>
>> > > the<br>
>> > ><br>
>> > > > confirm button, on an Ignored Face will un-mark the face as Ignored, and<br>
>> > > > move it to Unknown.<br>
>> > > ><br>
>> > > > Changes made to directories:<br>
>> > > > core/app<br>
>> > > > core/utilities<br>
>> > > > core/libs<br>
>> > > ><br>
>> > > > To do:<br>
>> > > > 1. The people sidebar should display Ignored Faces at the bottom, as it<br>
>> > > > isn't of priority to the user.<br>
>> > > > 2. Add Ignored Property to the metadata of the image, as discussed<br>
>> > ><br>
>> > > earlier.<br>
>> > ><br>
>> > > > Regards<br>
>> > > > Kartik<br>
>><br>
>><br>
>><br>
>><br>
</blockquote></div>