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

--- Comment #153 from Minh Nghia Duong <minhnghiaduong...@gmail.com> ---
(In reply to caulier.gilles from comment #151)
> Marcel,
> 
> You lost memory a little bit, but the infamous crash in Preview thread
> disappear. Right ?
> 
> If yes, we will trying to discover the source of the problem for this
> memory. Next stage for me is to add a new compilation option to use ASAN
> tool:
> 
> https://en.wikipedia.org/wiki/AddressSanitizer
> 
> About multithread while face detection, i'm not sure. If i remmeber, Nghia
> report to me that multicore support in face detection have been disabled for
> performance reasons, as low level API is not re-entrant.
> 
> Perhaps Nghia can provide more details for this point.
> 
> Gilles

Hi Gilles,

It's not quite what you said. I disabled multiple face extractors in faces
engine, which means for now, in faccial recognition, there is only one
FaceExtractor being used. This avoid multiple memory allocation for faical
recognition. However, with only one FaceExtractor, the facial recognition
process still runs multi-threading by using cv::parallel_for_ function.

It's implemented in file :
https://invent.kde.org/graphics/digikam/-/blob/master/core/libs/facesengine/recognition/opencv-dnn/opencvdnnfacerecognizer.cpp
, at lines 465 and 505. 

If you want to desactivate it for a test run, just replace these line with a
normal loop.

Nghia

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

Reply via email to