Skip to content

Conversation

@adradr
Copy link

@adradr adradr commented Aug 30, 2024

First of all thanks for this upscaler utility app, I really like what you're building also code-wise it's been quite easy to implement changes as its really clean and organized 🙌🏻

This PR is just a small modification as I was having issues when trying to upscale TV series all at once. Since the frames are contained and not deleted after the video processing completes the disk space was exploding and wanted to add a checkbox to allow automatic removal of the folder that holds extracted and upscaled frame images.

Take a look and feel free to merge if you like the idea! I compiled an executable for myself but would be great to use your releases in the future instead of my custom compile.

✨ Enhancements to Temporary File Management & User Preferences ✨
🔧 Added Functionality: Introduced a new option to delete temporary frames after upscaling is complete. This feature is now configurable through the GUI and the user preferences.
💾 User Preferences: Updated the system to remember the user's choice for deleting frames across sessions, stored within default_delete_frames.
🧹 Clean-Up Process: Implemented the delete_temporary_files function to recursively remove the entire directory, ensuring that temporary files are efficiently cleaned up.
🎛️ GUI Integration: Added a checkbox for the new delete option in the settings interface.

✨ Enhancements to Temporary File Management & User Preferences ✨
🔧 Added Functionality: Introduced a new option to delete temporary frames after upscaling is complete. This feature is now configurable through the GUI and the user preferences.
💾 User Preferences: Updated the system to remember the user's choice for deleting frames across sessions, stored within default_delete_frames.
🧹 Clean-Up Process: Implemented the delete_temporary_files function to recursively remove the entire directory, ensuring that temporary files are efficiently cleaned up.
🎛️ GUI Integration: Added a checkbox for the new delete option in the settings interface.
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

Successfully merging this pull request may close these issues.

1 participant