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

[Feature Request] Specify default working directory #12

Closed
Avasam opened this issue Feb 4, 2022 · 5 comments · Fixed by #13
Closed

[Feature Request] Specify default working directory #12

Avasam opened this issue Feb 4, 2022 · 5 comments · Fixed by #13
Labels
enhancement New feature or request

Comments

@Avasam
Copy link
Contributor

Avasam commented Feb 4, 2022

I would like to be able to specify a working directory for Unload. Here's why:

  1. Stay organized. Wherever I take the video from (either from my downloads folder, or my local recordings), the folder of frames will always be in a predetermined location that won't pollute other folders.
  2. Drive efficiency. My downloads and local recordings are on my 2TB harddrive. Aka my storage drive. When I load a video into Unload, it'll most likely be from either of those locations, and will create the frames on my slow storage drive, rather than my much preferable M.2 SSD. This is currently a huge bottleneck for me in terms of efficiency.
    image

Of course I can just move the video over to my SSD (let's say on my desktop). But that's more annoying steps, slowers, and doesn't help with staying organized. While specifying a working directory is a once time thing.

As a bonus: the saved csv file could also default to the specified folder location, with the same default filename as the video (just like the frames folder does)

@milankarman
Copy link
Owner

Personally I'm a big fan of having the frames folder right next to the video folder as for mass verification I just want to download them, check them and delete them right after - but I get why people might want this so I'd probably make it optional and off by default. Not sure when I'll get to it though. Thanks for the suggestion.

@milankarman milankarman added the enhancement New feature or request label Feb 5, 2022
@milankarman milankarman added this to To do in Unload 1.3.0 Feb 5, 2022
@Avasam
Copy link
Contributor Author

Avasam commented Feb 5, 2022

This is easy enough to do. I went ahead and started implementing it so you don't have to. I should be done by the end of the day.
image

@milankarman
Copy link
Owner

Nice! A pull request would be very welcome

@milankarman milankarman moved this from To do to In progress in Unload 1.3.0 Feb 7, 2022
@milankarman
Copy link
Owner

I changed the UI for this a bit to hide it below a settings button in 8c26723

XJyWklEYAL

@Avasam
Copy link
Contributor Author

Avasam commented Feb 10, 2022

I changed the UI for this a bit to hide it below a settings button in

Looks great! Definitely satisfies my needs.
Feel free to move "pick loads" back up

@milankarman milankarman moved this from In progress to Done in Unload 1.3.0 Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Unload 1.3.0
Ready for release
Development

Successfully merging a pull request may close this issue.

2 participants