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

Progress-bar keeps in Indeterminate mode #51

Closed
mjpmme opened this issue Apr 13, 2020 · 7 comments · Fixed by #57
Closed

Progress-bar keeps in Indeterminate mode #51

mjpmme opened this issue Apr 13, 2020 · 7 comments · Fixed by #57
Labels
is: enhancement New feature or extension of existing functionality
Projects

Comments

@mjpmme
Copy link

mjpmme commented Apr 13, 2020

Hi,
Thanks for doing this so awesome project. I'm not sure if it's an issue or something I misunderstand. Hopefully I make it here right.

I'm running it in docker. When a video is downloading, the progress bar doesn't show a percent complete, just keep looping from left to right. I tested it on Safari, Firefox and Chrome on macOS. I found it that this's called Indeterminate mode on this link: https://material.angular.io/components/progress-bar/overview.

How can I set it to show Determinate mode?

@Tzahi12345
Copy link
Owner

Glad you're enjoying it! :)

The indeterminate progress bar is intentional, but it wasn't always that way. I decided to make it so many versions ago because I had a hard time actually determining the progress on the file.

Basically, the client periodically asked the server to check on the file size of the downloading mp3/mp4 and compare it to the expected file size. It worked in most cases, but sometimes the file would split into parts or be in an entirely different format before being converted. So I gave up on the half-solution, and haven't revisited it since then.

But I was thinking about this recently, and it is something I'd like to implement sometime soon, as it would provide essential information to users. I'm sure there will be a lot of edge cases, so maybe have the indeterminate mode be a fallback in case the server can't determine the actual progress.

I'll put this up as a feature to add for the next update, v4.0. Still deciding which features I want for that update, so it might be some time until it gets implemented. Btw, you're in the right place for suggestions/questions, so feel free to open up any more issues if you encounter anything else or want to recommend a feature. They are always appreciated!

@Tzahi12345 Tzahi12345 added the is: enhancement New feature or extension of existing functionality label Apr 13, 2020
@Tzahi12345 Tzahi12345 added this to To do in v4.0 Apr 13, 2020
@mjpmme
Copy link
Author

mjpmme commented Apr 14, 2020

Great explanation for me. Thank you.

@nebula-it
Copy link

Just to add on to this, possible for progress bar to show up on other devices? E.g I start the download from my phone and opening up ytdl on my PC currently does not show currently downloading files/progress bar. It would be ideal if its possible to check from other devices as well.
Cheers

@Tzahi12345
Copy link
Owner

possible for progress bar to show up on other devices?

This is a good idea, but it will require a refactor of the way current downloads are managed. Right now it's all done in the UI, but for other devices to get that information the backend needs to be doing most of the legwork. This would be a great change actually, as it would give the backend more freedom on managing active downloads (making cancelling downloads actually possible, developing a possible queuing system)

I'll include this in the next update, thanks for the suggestion!

@Tzahi12345 Tzahi12345 moved this from To do to Done in v4.0 Apr 23, 2020
@Tzahi12345 Tzahi12345 linked a pull request Apr 23, 2020 that will close this issue
@Tzahi12345
Copy link
Owner

Tzahi12345 commented Apr 23, 2020

Just leaving a comment to let you know functional download progress bars has been implemented! It isn't enabled for audio files as the download speeds appeared significantly slower, but I will investigate this more and may make the downloading method configurable. Thanks @mjpmme for the suggestion!

@nebula-it your idea for viewing downloads across other devices is implemented as well. There's a new downloads manager that has been added where you can view all of this. It isn't organized in the prettiest way now so that will be worked on until it gets released in 4.0. Thank you for recommending this feature!

You can view the relevant pull request here for more info and a sneak peek of the downloads manager.

@nebula-it
Copy link

That looks great!. Good job @Tzahi12345 .
Looking forward to further enhancements with audio and playlist downloads.
Cheers

@mjpmme
Copy link
Author

mjpmme commented Apr 23, 2020

Thanks @Tzahi12345 that’s awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: enhancement New feature or extension of existing functionality
Projects
No open projects
v4.0
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants