search

OpenGenus / vidsum

Generate summary of any video :tv: anywhere and anytime
GNU General Public License v3.0
262 stars 69 forks source link

Doesn't load from YouTube #1

Closed TurnrDev closed 7 years ago

TurnrDev commented 7 years ago

I run python sum.py -u "https://www.youtube.com/watch?v=ZlXta87qyxU" as suggested in your wiki.. Error I get is..

usage: Watch videos quickly [-h] -i VIDEO_FILE -s SUBTITLES_FILE [-u URL]
Watch videos quickly: error: the following arguments are required: -i/--video-file, -s/--subtitles-file
AdiChat commented 7 years ago

Thanks for pointing out the issue. 👍

The solution is to make the required parameters optional.

TurnrDev commented 7 years ago

Damn you guys are fast are fixing bugs! Hats off to you! I just pulled the repo but now encountered this:

[jay@desktop-arch code]$ python sum.py -u "https://www.youtube.com/watch?v=ZlXta87qyxU"
  File "sum.py", line 99
    print url
            ^
SyntaxError: Missing parentheses in call to 'print'
py

Made some adjustments #7

lwgray commented 7 years ago

hi @JayTurnr Sorry, that 'print' statement was left there by accident. Corrected it with PR #8

shriakhilc commented 7 years ago

This issue seems to have been fixed, but it isn't closed. Is there something left to be done?

TurnrDev commented 7 years ago

@lwgray I fixed that in PR #7 but nevermind, take the blame. @TheGamer007 no, sorry I forgot to close