Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remaining temp files #279

Open
HolyGit12 opened this issue Jan 23, 2025 · 4 comments
Open

Remaining temp files #279

HolyGit12 opened this issue Jan 23, 2025 · 4 comments

Comments

@HolyGit12
Copy link

crf-search does not delete all temp files after finishing the search.
The both files of the last sample are staying in the temp folder. The temp folder won't be removed.
(Windows version but I guess that this does not matter)

@9Oc
Copy link

9Oc commented Jan 29, 2025

I am also experiencing this after the newest version [0.9.1] with the sample-encode command as well.

@alexheretic
Copy link
Owner

It's possible a recent ffmpeg process is still alive and prevents cleanup. I recently removed logic that would, a little too eagerly, kill ffmpeg so this makes sense.

So I'll look at adding graceful shutdown of these child processes which should resolve this.

@alexheretic
Copy link
Owner

I've implemented awaiting for child processes, like ffmpeg, to exit before temp file cleanup. Hopefully this should resolve this issue, though I couldn't actually reproduce it myself.

If possible can you let me know if this resolves the issue, you can install the latest code via cargo.

@mr44er
Copy link

mr44er commented Mar 2, 2025

Windows version but I guess that this does not matter

Maybe so. Windows Explorer starts generating thumbnails from videos and this means that a process hangs on the files that prevents them from being deleted. I just don't know how long the timeout is.
https://www.windowscentral.com/how-disable-thumbnails-windows-10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants