[digikam] [Bug 374225] Add possibility to remove face identities by removing tag [patch]

2017-01-10 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374225

Mario Frank  changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #4 from Mario Frank  ---
Hi Simon,

I would go even further. Removing the identity when deleting the tag is
definitely a good idea. But I see another use-case that hit me multiple times.
I assigned a tag as face erroneously to some image. But removing the face from
the image did not delete the ImageProperty that states that this tag is a
person tag. The bug https://bugs.kde.org/show_bug.cgi?id=262168 is somehow
connected to this.

Thus, deleting a person tag or a list of person tags but preserving them as
ordinary tags is helpful to clean the DB. Otherwise, users would have to dig
into the DB data which is not really nice for most of them.

I introduced a new context menu item "Delete People Tag" which works for
singleton people tags and also for lists.
This action removes the autodetected face and region property for all images
for the people tags and if wished (by confirmation) also the connection between
the tag and the image, i.e. untaggs the image.
Also, the person, faceEngineId and faceEngineUuid property of the tags is
removed and if existent, the Identity from the recognition.db. The recognition
db entry is found by the uuid.

By the way, I had some TagProperties entries with property kfaceId and
kfaceUuid, but did not find that in source code. May it be that the properties
were renamed?

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374225] Add possibility to remove face identities by removing tag [patch]

2016-12-27 Thread Simon
https://bugs.kde.org/show_bug.cgi?id=374225

--- Comment #3 from Simon  ---
> --- Comment #2 from caulier.gil...@gmail.com ---
> Why forgot the patch ???
>
> Gilles Caulier
>
Sorry, that was misleading, everything is fine.
I forgot to add the patch to the initial comment, the same as with the
usual follow up email when you forget your attachments. However with the
KDE bug tracker it is normal to add attachments in an additional
comment, so my remark was completely unnecessary.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374225] Add possibility to remove face identities by removing tag [patch]

2016-12-27 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=374225

caulier.gil...@gmail.com changed:

   What|Removed |Added

 CC||caulier.gil...@gmail.com
Summary|Add possibility to remove   |Add possibility to remove
   |face identities by removing |face identities by removing
   |tag |tag [patch]

--- Comment #2 from caulier.gil...@gmail.com ---
Why forgot the patch ???

Gilles Caulier

-- 
You are receiving this mail because:
You are watching all bug changes.