Bug 384593 - Preview optimization: Dont rerender chunks
Summary: Preview optimization: Dont rerender chunks
Status: REPORTED
Alias: None
Product: kdenlive
Classification: Applications
Component: Video Display & Export (show other bugs)
Version: git-master
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-09-11 12:46 UTC by alberto
Modified: 2017-09-29 15:53 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description alberto 2017-09-11 12:46:50 UTC
Using spacer tool doenst invalidate the preview as per #367079 but invalidating it after manually doesnt seem to be very optimized either. It repreview everything again

It doesnt recognize that the frame has been rendered before just because it moved in time

I guess a md5 of the parameters used to invalidate a clip discarding the time (looking at the file i think is called <property name="length">) would fix this. After this, theres probably no need to save them in preview.mlt like is done right now either
Comment 1 alberto 2017-09-11 15:02:50 UTC
The lenght is probably the frame number and thats what is probably whats used now. It it doesnt match with the second evaluated (which is very likely not matching when you move clips with so many fps) will re-render that chunk and every chunk afterwards


Maybe calculate first if the "first frame + number of fps < chunk already rendered" and re-render only up to frames that are missing. This way you can use everything afterwards, even if the first chunk uses 2 previews videos instead of 1

And only after preview engie goes idle, redo 1 second chunks that are made out of more than 3-5 videos as maintenance, to avoid growing the number of videos exponentially while having a low impact in the editing process :)

Thanks for listening!

Not sure if the new timeline work that is being carried out now will fix all these bugs. Please let me now if they do :)
Comment 2 alberto 2017-09-14 16:51:20 UTC
Hidding invalidates the preview, which not exactly wrong, but unhiding it, doesnt recover the chunks already rendered, despite not having changed anything at all

Preview is an awesome feature and have a lot of potential, it just need to use a few optimizations here and there

Mostly dont renrender things that havent change and use all cores to render different chunks

I change the importance of the bug to wishlist, since nothing is really broken because of this. Its just very slow and you have to be very careful which what you touch when editing knowning you are going to lose the preview speed with certain moves :)

Thanks again for this awesome editor!