SUMMARY There's a ton of video import temp pngs left behind when Krita crashes or is OOM-Killed during video import. STEPS TO REPRODUCE 1. Select a large amount of frames from a video file. Too many, infact. 2. Import them all. (or just kill Krita from the terminal while it imports) 3. Watch Krita kill itself. 4. Check the folder of the video file to find a ton of temp files. I have no idea how we'd really fix this, outside of maybe only extracting a small number of frames at a time and importing those first before heading towards the next set.
The solution here would be to do frame extraction in batches and also restrict the number of frames. I might also look into ways to see if ffmpeg can be smart about skipping frames that are equivalent to a certain threshold...
Looking into this more, having now optimized the FFMPEG to import less frames (optionally) for longer files on my own branch, another solution is to properly use the temp directories at krita's disposal. At least on linux, these temp directories will be cleaned up after system reboot.
Git commit b6b1f2355e0118f23c39a002273f0ad1b1c9edfa by Eoin O'Neill. Committed on 06/01/2022 at 23:23. Pushed by eoinoneill into branch 'master'. Initial proof-of-concept of keyframe detection and duplicate frame removal during video import process. Steps remaining... 1) Plug UI in with meaningful value range for users to tweak. 2) Use frame file names to determine position within file, and insert accordingly. ... And more? This should help people when importing animations that are long and repetitive. They might not need every single duplicate frame, depending on their user case. Users will be able to dial in sensitivity to adjust to their liking. M +3 -2 libs/ui/animation/KisDlgImportVideoAnimation.cpp M +102 -37 libs/ui/animation/VideoImportDialog.ui https://invent.kde.org/graphics/krita/commit/b6b1f2355e0118f23c39a002273f0ad1b1c9edfa
Git commit 86db9358f2fea2b528346a5f7351dcbe58948316 by Eoin O'Neill. Committed on 11/01/2022 at 21:55. Pushed by eoinoneill into branch 'master'. Use QTemporaryDir class instead of manual temporary directory management. Should help w/ preventing file leakage. In odd case that it does crash on import, QTemporaryDir should safely clean up files (and, if not, files should at least be cleaned on reboot.) M +5 -5 libs/ui/KisMainWindow.cpp M +5 -15 libs/ui/animation/KisDlgImportVideoAnimation.cpp M +1 -4 libs/ui/animation/KisDlgImportVideoAnimation.h https://invent.kde.org/graphics/krita/commit/86db9358f2fea2b528346a5f7351dcbe58948316