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

Use PagedList in YoutubeFeedsInfoExtractor #10184

Closed
zamanisaja opened this issue Jul 29, 2016 · 2 comments
Closed

Use PagedList in YoutubeFeedsInfoExtractor #10184

zamanisaja opened this issue Jul 29, 2016 · 2 comments
Labels
bug

Comments

@zamanisaja
Copy link

@zamanisaja zamanisaja commented Jul 29, 2016

$ youtube-dl --netrc :ytsubs --playlist-end 20 --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--netrc', u':ytsubs', u'--playlist-end', u'20', u'--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.07.26.2
[debug] Python version 2.7.6 - Linux-3.16.0-38-generic-x86_64-with-LinuxMint-17.2-rafaela
[debug] Proxy map: {}
[youtube:subscriptions] Downloading login page
[youtube:subscriptions] Logging in
[youtube:subscriptions] Youtube Subscriptions: Downloading webpage
[youtube:subscriptions] Youtube Subscriptions: Downloading page #1
[youtube:subscriptions] Youtube Subscriptions: Downloading page #2

and it just keeps going on ...
what am i doing wrong?!

@yan12125
Copy link
Collaborator

@yan12125 yan12125 commented Jul 31, 2016

The current implementation downloads the whole playlist and then fetch the first 20 videos. It's possible and should be just download the first few pages.

@yan12125 yan12125 added the bug label Jul 31, 2016
@yan12125 yan12125 changed the title weird problem with :ytsubs Use PagedList in YoutubeFeedsInfoExtractor Jul 31, 2016
@klaas
Copy link

@klaas klaas commented Mar 28, 2018

+1 for this one!

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
3 participants
You can’t perform that action at this time.