Commit Graph

7293 Commits (ab84349b16b3c94775543a04855fc77005f8237e)
 

Author SHA1 Message Date
Jaime Marquínez Ferrándiz ab84349b16 [test/YoutubeDL] Add test for subtitles
Updated the offlinetest make target to not skip it
9 years ago
Jaime Marquínez Ferrándiz 03091e372f [ted] Always extract the subtitles
The required info is already in the webpage
9 years ago
Jaime Marquínez Ferrándiz 4d17184817 [YoutubeDL] don't set the 'requested_subtitles' without writesubtitles or writeautomaticsub 9 years ago
Jaime Marquínez Ferrándiz fb7cb6823e Remove the SubtitlesInfoExtractor class
No longer needed
9 years ago
Jaime Marquínez Ferrándiz f13b1e7d7f [bbccouk] Convert to new subtitles system
I haven't found any video available outside the UK, so I haven't added tests.

I have updated how the srt file is build, because (at least for www.bbc.co.uk/programmes/p02j9b69) the subtitles is inside 'span' elements.
9 years ago
Jaime Marquínez Ferrándiz 8807f1277f [theplatform] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz 4f7cea6c53 [viki] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz afbdd3acc3 [rai] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz 01561da142 [nrk] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz 0af25f784b [mtv] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz b9b42f2ea0 [npo] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz 311c393838 [lynda] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz 18c1c42405 [drtv] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz 37dd5d4629 [mit] Don't set the subtitles field
YouTube already provides them in more formats
9 years ago
Jaime Marquínez Ferrándiz edab9dbf4d [YoutubeDL] use the 'render_table' function for listing the subtitles 9 years ago
Jaime Marquínez Ferrándiz 9868ea4936 [extractor/common] Simplify subtitles handling methods
Initially I was going to use a single method for handling both subtitles and automatic captions, that's why I used the 'list_subtitles' and the 'subtitles' variables.
9 years ago
Jaime Marquínez Ferrándiz 85920dd01d [bliptv] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz bd7fe0cf66 [walla] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz 48246541da [ceskatelevize] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz 360e1ca5cc [youtube] Convert to new subtitles system
The automatic captions are stored in the 'automactic_captions' field, which is used if no normal subtitles are found for an specific language.
9 years ago
Jaime Marquínez Ferrándiz a1f2a06b34 [dailymotion] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz c84dd8a90d [YoutubeDL] store the subtitles to download in the 'requested_subtitles' field
We need to keep the orginal subtitles information, so that the '--load-info' option can be used to list or select the subtitles again.
We'll also be able to have a separate field for storing the automatic captions info.
9 years ago
Jaime Marquínez Ferrándiz 65469a7f8b [vimeo] Convert to new subtitles system
Removed some tests, the behaviour should be checked in a test for the YoutubeDL class
9 years ago
Jaime Marquínez Ferrándiz 6b597516c1 [atresplayer] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz b5857f62e2 [crunchyroll] Convert to new subtitles system 9 years ago
Jaime Marquínez Ferrándiz a504ced097 Improve subtitles support
For each language the extractor builds a list with the available formats sorted (like for video formats), then YoutubeDL selects one of them using the '--sub-format' option which now allows giving the format preferences (for example 'ass/srt/best').
For each format the 'url' field can be set so that we only download the contents if needed, or if the contents needs to be processed (like in crunchyroll) the 'data' field can be used.

The reasons for this change are:
* We weren't checking that the format given with '--sub-format' was available, checking it in each extractor would be repetitive.
* It allows to easily support giving a format preference.
* The subtitles were automatically downloaded in the extractor, but I think that if you use for example the '--dump-json' option you want to finish as fast as possible.

Currently only the ted extractor has been updated, but the old system still works.
9 years ago
Jaime Marquínez Ferrándiz 8fb474fb17 [test/subtitles] Fix some tests
The checksym for the CeskaTelevize subtitles has changed again, so we just test that it has a reasonable length.
9 years ago
Sergey M․ f813928e4b [bbccouk] Fix fallback to legacy playlist 9 years ago
Sergey M․ b9c7a97318 [history] Add extractor (Closes #4934) 9 years ago
Sergey M․ 9fb2f1cd6d [theplatform] Add URL sign capability 9 years ago
Jaime Marquínez Ferrándiz 6ca7732d5e [extractor/common] Fix link to external documentation 9 years ago
Jaime Marquínez Ferrándiz b0ab0fac49 Remove unused imports 9 years ago
Naglis Jonaitis a294bce82f [streamcz] Fix extraction (Closes #4940) 9 years ago
Sergey M․ 76d1466b08 [drtuber] Add one more title regex 9 years ago
Sergey M. 1888d3f7b3 Merge pull request #4951 from peugeot/beeg
[beeg] fix test
9 years ago
Sergey M. c2787701cc Merge pull request #4950 from peugeot/drtuber
[drtuber] fix extraction
9 years ago
peugeot 52e1d0ccc4 [beeg] fix test 9 years ago
peugeot 10e3c4c221 [drtuber] fix extraction 9 years ago
Sergey M․ 68f2d273bf [sunporno] Keep old video regex just in case 9 years ago
Sergey M. 7c86c21662 Merge pull request #4949 from peugeot/sunporno
[sunporno] fix extraction
9 years ago
peugeot ae1580d790 [sunporno] fix extraction 9 years ago
Sergey M․ 3215c50f25 Credit @ryandesign for nbcnews nightly news (#4948) 9 years ago
Sergey M․ 36f73e8044 Merge branch 'ryandesign-nbc-nightly-news' 9 years ago
Sergey M․ a4f3d779db [nbcnews] Simplify 9 years ago
Ryan Schmidt d9aa2b784d Support NBC Nightly News broadcasts 9 years ago
Jaime Marquínez Ferrándiz cffcbc02de [postprocessor/ffmpeg] Don't let ffmpeg read from stdin (fixes #4945)
If you run 'while read aurl ; do youtube-dl --extract-audio "${aurl}"; done < path_to_batch_file'  (batch_file contains one url per line) each call to youtube-dl consumed some characters and 'read' would assing to 'aurl' a non valid url, something like 'tube.com/watch?v=<id>'.
9 years ago
Sergey M․ 9347fddbfc [1tv] Cover arbitraty URLs 9 years ago
Sergey M․ 037e9437e4 [camdemy] Fix _VALID_URL 9 years ago
Jaime Marquínez Ferrándiz 36e7a4ca2e [test/subtitles] Update checksums 9 years ago
Jaime Marquínez Ferrándiz ae6423d704 [bambuser] Fix 'uploader_id' extraction (fixes #4944) 9 years ago