https://bugs.kde.org/show_bug.cgi?id=395241

--- Comment #33 from Michael Miller <michael_mil...@msn.com> ---
(In reply to Michael Moore from comment #32)
> > [Mike] If you're willing to try very unstable code, grab the latest 
> > AppImage and let's see if we can reproduce the issue.  If we can, then it 
> > has to be something in the thumbnail cache. You can find it here: 
> > https://files.kde.org/digikam/
> 
> I'm OK with unstable code. My photos are safely backed up. 
Awesome.  Please let us know how it goes.

> > [Mike] I can tell you with certainty that in the 8.6.0 pipelines the 
> > thumbnail is written to the DB when it is first extracted from the original 
> > image during face detection.
> > [Maik] Incorporating the creation of thumbnails in the face scan process is 
> > also not a good idea, because we might then create the thumbnails twice. 

Maik, we can easily remove thumbnail creation from the 8.6.0 pipelines if you
think that's best.  My thoughts, though, are the most expensive operation is
loading the original image (especially RAW images), which I need to do in the
pipeline to detect the faces.  Then, the face detector gives me the face region
coordinates, so we already have them.  The only step that's missing is saving
the thumbnail to the DB, which is relatively quick in comparison. The 8.6.0
pipelines are the same speed as 8.5.0 for single threading, and much faster
when using full CPU so there isn't a performance penalty for creating the
thumbnails when we detect faces. If we compare the image modification timestamp
with the thumbnail timestamp we can quickly determine if we need to rebuild the
thumbnail from the original image.  I'm thinking this could speed up loading
face thumbnails.

> In any case, I guess that faces which are already detected won't have
> thumbnails automatically generated. 
> 
> If thumbnails are in fact supposed to be generated somewhere in the
> detection process, how can I test it and report back here? Do I need to do
> the Maintenance item "Reset and clear all faces and training" and then scan
> them again to test if they are generated at some point after detection? 
Yes, clear all faces and rescan.  The 8.6.0 face scanning has been simplified
so that "Scan new images" and "Scan all images" also does recognition
(matching).

Cheers,
Mike

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

Reply via email to