-
Notifications
You must be signed in to change notification settings - Fork 205
Add Transcript & Channel class #165
Conversation
alexmercerind
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me! 💚
Well, we surely have a lot of features in this library.
raitonoberu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works well, great job! Just a couple of comments:
- If subtitles are disabled in the video, it raises the
Failed to extract keyexception. It would be nice to add a proper way to handle this. Maybe just return an empty dict? - Now it returns transcriptions in English. How about adding a language selection option?
This I will certainly fix, but I haven't got an example of video like this.
Will have a look. |
Mostly various music videos (not sure how it works): https://www.youtube.com/watch?v=hzpt3fQjY9U |
|
Done. |
|
I've also added Channel class, which displays channel info - Resolves #164 |
|
Will merge this tomorrow |
So, YouTube has added transcription (slovenian "Prepis"). Now we have Transcription classes with really minimal code.

Have added tests, but adding you as reviewers as you never know what you can find 😉.