berthold.stoeger at tuwien.ac.at
Fri Jul 6 07:36:21 PDT 2018
I would just calculate one thumbnail at a time. But just go wild and I can try to marry our solutions later, if needed.
Sending via webmail, so the mailinglist will probably reject.
From: Robert Helling [helling at lmu.de]
Sent: Friday, July 6, 2018 3:38 PM
To: Stöger, Berthold
Cc: Dirk Hohndel; subsurface at subsurface-divelog.org
Subject: Re: Video thumbnails
> On 6. Jul 2018, at 15:32, Berthold Stoeger <bstoeger at mail.tuwien.ac.at> wrote:
> We already do all that. And the code should also recognize if thumbnails are
> older than the file. So I'm not sure if implementing a parallel thumbnail-
> system for videos is a UI improvement.
>> and create image files there.
>> From there, Subsurface could look for such a directory and then display the
>> thumbnails as images (at the given time intervals). So the idea would be to
>> trade disk space (people who have videos should have plenty of that) for
>> runtime generating the thumbnails on the fly for each run of Subsurface.
>> What do you think?
> Can't ffmpeg write the image to stdout? Then we could just read it in a thread
> and send it via signal/slot to the thumbnailing system. I always prefer "push"
> over "pull" interfaces.
I don’t know about writing to stdout, maybe. But still those would be several, wouldn’t it? What do you think is a good rate, one image per minute?
Let me modify my proposal: Trigger the generation upon opening the dive (and not having thumbnails) in a background task. Store the images in a temporary directory. Upon completion of the ffmpeg process add those images to the Subsurface thumbnails. Does that sound better (that would be a signal based push if you like).
Robert C. Helling Elite Master Course Theoretical and Mathematical Physics
Ludwig Maximilians Universitaet Muenchen, Dept. Physik
Phone: +49 89 2180-4523 Theresienstr. 39, rm. B339
Enhance your privacy, use cryptography! My PGP keys have fingerprints
A9D1 A01D 13A5 31FA 6515 BB44 0820 367C 36BC 0C1D and
DCED 37B6 251C 7861 270D 5613 95C7 9D32 9A8D 9B8F
More information about the subsurface