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

Error in downloading video: global name 'speed_str' is not defined #1731

Closed
darth10 opened this issue Nov 6, 2013 · 7 comments
Closed

Error in downloading video: global name 'speed_str' is not defined #1731

darth10 opened this issue Nov 6, 2013 · 7 comments

Comments

@darth10
Copy link

@darth10 darth10 commented Nov 6, 2013

I'm getting this error with the latest build on Windows 7.
I obtained the build from the downloads page.
The command I ran was youtube-dl <URL>.

[youtube] Setting language
[youtube] HXfDK1OYpco: Downloading video webpage
[youtube] HXfDK1OYpco: Downloading video info webpage
[youtube] HXfDK1OYpco: Extracting video information
[download] Resuming download at byte 788
[download] Destination: Clojure  - Deep Walking Macros-HXfDK1OYpco.mp4
Traceback (most recent call last):
  File "__main__.py", line 18, in <module>
  File "youtube_dl\__init__.pyo", line 766, in main
  File "youtube_dl\__init__.pyo", line 719, in _real_main
  File "youtube_dl\YoutubeDL.pyo", line 715, in download
  File "youtube_dl\YoutubeDL.pyo", line 363, in extract_info
  File "youtube_dl\YoutubeDL.pyo", line 444, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 388, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 477, in process_video_result
  File "youtube_dl\YoutubeDL.pyo", line 688, in process_info
  File "youtube_dl\FileDownloader.pyo", line 553, in _do_download
NameError: global name 'speed_str' is not defined

Here's the version from youtube-dl -v.

[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-v']
[debug] youtube-dl version 2013.11.03
[debug] Python version 2.7.5 - Windows-7-6.1.7600
[debug] Proxy map: {}
Usage: youtube-dl.exe [options] url [url...]
@phihag phihag closed this in 50a886b Nov 6, 2013
@phihag
Copy link
Contributor

@phihag phihag commented Nov 6, 2013

Thank you for the report. I believe this issue to be fixed in youtube-dl 2013.11.06 and newer. You can simply type youtube-dl -U to update.

I'm closing this issue now, but if the problem occurs, feel free to comment here and we'll immediately reopen it. Thanks again!

@darth10
Copy link
Author

@darth10 darth10 commented Nov 7, 2013

I updated using the -U option, but now I get a different error. Is this an existing issue?

> youtube-dl.exe http://www.youtube.com/watch?v=HXfDK1OYpco
[youtube] Setting language
[youtube] HXfDK1OYpco: Downloading video webpage
[youtube] HXfDK1OYpco: Downloading video info webpage
[youtube] HXfDK1OYpco: Extracting video information
[download] Resuming download at byte 788
[download] Destination: Clojure  - Deep Walking Macros-HXfDK1OYpco.mp4
Traceback (most recent call last):
  File "__main__.py", line 18, in <module>
  File "youtube_dl\__init__.pyo", line 766, in main
  File "youtube_dl\__init__.pyo", line 719, in _real_main
  File "youtube_dl\YoutubeDL.pyo", line 715, in download
  File "youtube_dl\YoutubeDL.pyo", line 363, in extract_info
  File "youtube_dl\YoutubeDL.pyo", line 444, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 388, in process_ie_result
  File "youtube_dl\YoutubeDL.pyo", line 477, in process_video_result
  File "youtube_dl\YoutubeDL.pyo", line 688, in process_info
  File "youtube_dl\FileDownloader.pyo", line 560, in _do_download
  File "youtube_dl\FileDownloader.pyo", line 237, in report_progress
  File "youtube_dl\FileDownloader.pyo", line 110, in format_eta
  File "youtube_dl\FileDownloader.pyo", line 75, in format_seconds
TypeError: unsupported operand type(s) for divmod(): 'str' and 'int'
jaimeMF added a commit that referenced this issue Nov 7, 2013
The report_progress function will accept eta and percent with None value and will set the message to 'Unknow ETA' or 'Unknown %'.
Otherwise the values must be numbers.
@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Nov 7, 2013

It shoudl be fixed now, you'll need to wait until the next release. Thanks for the report.

@darth10
Copy link
Author

@darth10 darth10 commented Nov 7, 2013

Oh I see. Alrighty then 👍

@phihag
Copy link
Contributor

@phihag phihag commented Nov 7, 2013

@jaimeMF Thanks for fixing this, I kinda rushed the code in between projects.

@darth10 You can now update to 2013.11.07, which should finally fix this. Feel free to report any other problems you encounter. Thanks for the update, and sorry for the delay.

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Nov 7, 2013

@phihag Don't worry, I broke it, so I had to fix it :).

@darth10
Copy link
Author

@darth10 darth10 commented Nov 7, 2013

Seems fixed! 👍
Great job guys!

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.