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

No ETA with 8KB/s bandwidth #1280

Closed
lightonflux opened this issue Aug 20, 2013 · 4 comments
Closed

No ETA with 8KB/s bandwidth #1280

lightonflux opened this issue Aug 20, 2013 · 4 comments

Comments

@lightonflux
Copy link

@lightonflux lightonflux commented Aug 20, 2013

I have to use my phone to have internet from time to time. And sometimes the connections is lightning fast, more precisely ~8000B/s.

And yt-dl works fine, execpt that there is no ETA, for some reason.

 [download]  22.0% of 292.42MiB at  7.12KiB/s ETA --:-- 

I don't see a reason why yt-dl does not have an ETA, because it knows how big the file is and how fast the download is. So math should no be the problem.

The tool trickle (for linux) is nice to test it, if you are in the sad situation and have VDSL. :)

@phihag
Copy link
Contributor

@phihag phihag commented Aug 20, 2013

Currently, we're cutting off ETA display if it is more than 99 minutes. I agree that we should increase that to, say, 99 hours (which is more than 4 days).

@lightonflux
Copy link
Author

@lightonflux lightonflux commented Aug 21, 2013

Why limit at all? I don't see a reason for it.

@phihag
Copy link
Contributor

@phihag phihag commented Aug 21, 2013

Because an output like ETA: 3 millenia (or ETA: 26282953 hours) is totally useless. The ETA is only an estimation, and if the value is that high, it's way more likely your connection has been lost than that the actual ETA is anywhere in that range.

@phihag phihag closed this in 6c3e6e8 Aug 21, 2013
@lightonflux
Copy link
Author

@lightonflux lightonflux commented Aug 21, 2013

Thanks for the explanation.

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

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.