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

Add support for passing %(duration)s to --sleep parameters #14721

Closed
JarLob opened this issue Nov 11, 2017 · 2 comments
Closed

Add support for passing %(duration)s to --sleep parameters #14721

JarLob opened this issue Nov 11, 2017 · 2 comments

Comments

@JarLob
Copy link

@JarLob JarLob commented Nov 11, 2017

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.11.06. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

  • [ x] I've verified and I assure that I'm running youtube-dl 2017.11.06

Before submitting an issue make sure you have:

  • [x ] At least skimmed through the README, most notably the FAQ and BUGS sections
  • [x ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

  • Bug report (encountered problems with youtube-dl)
  • Site support request (request for adding support for a new site)
  • [x ] Feature request (request for a new functionality)
  • Question
  • Other

In order to not get banned it would be logical to sleep for the duration of the last downloaded video in a playlist. There is already support for duration in Output Template, but it doesn't work when passed to --sleep-interval.
Please implement something similar to:

  1. "--sleep-interval %(duration)s"
  2. "--sleep-interval %(duration)s --max-sleep-interval %(duration + 123)s" The idea is to sleep exactly how long the video lasts or a bit longer.
@JarLob JarLob changed the title Add support of passing %(duration)s to --sleep parameters Add support for passing %(duration)s to --sleep parameters Nov 11, 2017
@dstftw
Copy link
Collaborator

@dstftw dstftw commented Nov 11, 2017

  1. duration is optional and may not be present.
  2. Output template does not support any calculations.
  3. Sleeping for duration time after downloading the whole file immediately does not make much sense since it still does not mimic webplayer behavior and easily detectable. Rate limiting download to video bitrate is more reasonable in this case.
@dstftw dstftw closed this Nov 11, 2017
@JarLob
Copy link
Author

@JarLob JarLob commented Nov 11, 2017

  1. Presence of "duration" is specific to extractor, same as a chance to get a Ban. Pluralsight, for example. I know there is a duration implemented in the extractor and I know there is a chance to be banned. So it would work for specific extractors.
  2. The second example with calculations was a "nice to have". At least feature 1. that doesn't require calculations could have been implemented.
  3. I'm new to youtube-dl. Could you provide an example what parameters to pass to stay least detectable? What to pass to youtube-dl to limit download rate to the video bitrate automatically?
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.