stemrollerapp / stemroller

Isolate vocals, drums, bass, and other instrumental stems from any song
https://www.stemroller.com
Other
2.62k stars 103 forks source link

Suggestion: Show progress instead of "Processing" #8

Closed alectrocute closed 1 year ago

alectrocute commented 2 years ago

Since this is a long-running process, it would be really helpful to print some sort of progress/stdout/etc. to the user. I'd be happy to make a PR for this if I have time, but am leaving this here just in case.

iffyloop commented 2 years ago

Great idea! Should be possible, as Demucs prints progress during processing files, just need to figure out a way to handle that stdout data...

iffyloop commented 1 year ago

After a short while attempting to parse data from the Demucs process and update the UI accordingly, it turns out this is a non-trivial issue that will require some refactoring (and also better understanding of how to stream the data from the Demucs child process). Would really like to have this feature implemented but am not planning to work on it myself anytime in the near future. Will leave the issue open though in case anyone else wants to give it a try or I have time to fix it later.

iffyloop commented 1 year ago

Devin Burnette's changes have been merged into the develop branch and should be merged into main shortly... closing.