<div dir="ltr"><div>Hi Maik,</div><div><br></div>Definitively no, because we need to limit the dependencies puzzle definitively. Look well, this thumbnailer use KIO in background, and you know what i mean about the portability of KIO...<div><br></div><div>The DK thumbnailer architecture is based on QThread, and we can use here ActionThreadBase/ActionJob instead QObject/Qthread.<br><div><br></div><div>But you can include your patch in future internal digiKam core video thumbnailer based on this kind of code (:=)))</div><div><br></div><div>Gilles</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-03-04 18:10 GMT+01:00 Maik Qualmann <span dir="ltr"><<a href="mailto:metzpinguin@gmail.com" target="_blank">metzpinguin@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Do we want to use libffmpegthumbnailer as additional dependencies? Also used<br>
by Dolphin here. For that I already have a nearly finished patch, because I<br>
tested it some time ago and it worked very well and fast. I will reactivate<br>
the patch to test...<br>
<span class="HOEnZb"><font color="#888888"><br>
Maik<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
Am Sonntag, 4. März 2018, 17:49:39 CET schrieb Gilles Caulier:<br>
> Hi all,<br>
><br>
> Since i written the FFMPEG video metadata parser in 6.0.0 branch, i'm<br>
> trying to register a huge video collection from my host computer.<br>
><br>
> Here, more than 2500 MKV files are present, plus a lots of MP4, MOV, and<br>
> another media taken with cellular and camera.<br>
><br>
> The registration have been very fast. 2500 file parsed in 2 minutes and<br>
> ready to use. System has SSD, Sqlite, 8 cores, and 32Gb of RAM. No crash<br>
> all work fine... excepted... the video thumbnails.<br>
><br>
> I facing random dysfunctions with video files. Sometime thumbs are there,<br>
> sometime not. Using F5 always give a different results until i try to<br>
> insist and i can see the DB locked and revival...<br>
><br>
> The video thumbnailer is based on QtAV frame extractor. My QtAV is last<br>
> stable 1.12.0. I suspect a bug somewhere in QtAV, as i can see thumbnailer<br>
> code including a lots of mutex to prevent concurrency.<br>
><br>
> This is typically abnormal. FFMPEG API is re-entrant as i checked with the<br>
> multi-threaded metadata parser unit test.<br>
><br>
> As we depend now directly of FFMPEG API, I propose to rewrite the video<br>
> thumbnailer with libav C function.<br>
><br>
> We have code ready to use with ffmpeg API here :<br>
><br>
> <a href="https://cgit.kde.org/ffmpegthumbs.git/tree/" rel="noreferrer" target="_blank">https://cgit.kde.org/<wbr>ffmpegthumbs.git/tree/</a><br>
><br>
> Your viewpoint ?<br>
><br>
> Gilles<br>
<br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>