Repository: yt-dlp/yt-dlp
Pull request #2359 (2022-01-16 15:36:56)

[TED] extractor rewrite
## Please follow the guide below

- You will be asked some questions, please read them **carefully** and answer honestly
- Put an `x` into all the boxes [ ] relevant to your *pull request* (like that [x])
- Use *Preview* tab to see how your *pull request* will actually look like

---

### Before submitting a *pull request* make sure you have:
- [x] At least skimmed through [contributing guidelines](https://github.com/yt-dlp/yt-dlp/blob/master/CONTRIBUTING.md#developer-instructions) including [yt-dlp coding conventions](https://github.com/yt-dlp/yt-dlp/blob/master/CONTRIBUTING.md#yt-dlp-coding-conventions)
- [x] [Searched](https://github.com/yt-dlp/yt-dlp/search?q=is%3Apr&type=Issues) the bugtracker for similar pull requests
- [x] Checked the code with [flake8](https://pypi.python.org/pypi/flake8)

### In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under [Unlicense](http://unlicense.org/). Check one of the following options:
- [x] I am the original author of this code and I am willing to release it under [Unlicense](http://unlicense.org/)
- [ ] I am not the original author of this code but it is in public domain or released under [Unlicense](http://unlicense.org/) (provide reliable evidence)

### What is the purpose of your *pull request*?
- [x] Bug fix
- [ ] Improvement
- [ ] New extractor
- [ ] New feature

---

### Description of your *pull request* and other information

TED has restructured their site and a rewrite for TED extractor is required (#2343).

changes:
- rewrite talks and playlist handling
- added series handling
- removed handling /watch/ urls (was very old code and new watch pages no longer contains a single video as the old extractor expects)

I have used quite a bit of the old parsing code as some of them still matches the new json structure. There are parts of code which I am not sure if they are still necessary with their new player (e.g. audio_download and rtmp source), but since they are not fatal I chose to leave them as is.

pukkandan: Add some tests for playlist, series and #season
trassshhub: I was trying to use series_id instead of id, it works with the binary but the test is complaining that the id key is not present.
Is there a way around this or shall I just leave it to use id?
Ashish0804: This was a partial review.
Remove useless/obvious comments and add `or <empty object>` after every try_get and .get used to get iterables. 

trassshhub: Thanks for the review, I have committed the suggested changes.

by the way, for this issue I mentioned above, it may not be directly related to this PR but I want some insights from the reviewers.

> I was trying to use series_id instead of id, it works with the binary but the test is complaining that the id key is not present. Is there a way around this or shall I just leave it to use id?
> 
> ```
> Traceback (most recent call last):
>   File "P:\BT\software\yt-dlp\test\test_download.py", line 191, in test_template
>     expect_info_dict(self, res_dict, test_case.get('info_dict', {}))
>   File "P:\BT\software\yt-dlp\test\helper.py", line 257, in expect_info_dict
>     test_info_dict = sanitize_got_info_dict(got_dict)
>   File "P:\BT\software\yt-dlp\test\helper.py", line 238, in sanitize_got_info_dict
>     if test_info_dict.get('display_id') == test_info_dict['id']:
> KeyError: 'id'
> ```

common.py states that playlist 'can' have 'id' which means it is not compulsory.
https://github.com/yt-dlp/yt-dlp/blob/32b95bb6436fd17ee21cd55217f16361d4e7494a/yt_dlp/extractor/common.py#L385-L395

But the test helper seems to be treating 'id' as compulsory and therefore using test_info_dict['id'] and not test_info_dict.get('id')?
https://github.com/yt-dlp/yt-dlp/blob/32b95bb6436fd17ee21cd55217f16361d4e7494a/test/helper.py#L237-L239

Which should be the correct behavior?
pukkandan: Probably an oversight. I'll look into it. Playlist Id is optional

Patch:
diff --git a/yt_dlp/extractor/ctvnews.py b/yt_dlp/extractor/ctvnews.py
index 03f8cefb7..952f4c747 100644
--- a/yt_dlp/extractor/ctvnews.py
+++ b/yt_dlp/extractor/ctvnews.py
@@ -65,4 +65,9 @@ def ninecninemedia_url_result(clip_id):
             })
             entries = [ninecninemedia_url_result(clip_id) for clip_id in orderedSet(
                 re.findall(r'clip\.id\s*=\s*(\d+);', webpage))]
+            if not entries:
+                webpage = self._download_webpage(url, page_id)
+                if 'getAuthStates("' in webpage:
+                    entries = [ninecninemedia_url_result(clip_id) for clip_id in
+                               self._search_regex(r'getAuthStates\("([\d+,]+)"', webpage, 'clip ids').split(',')]
             return self.playlist_result(entries, page_id)
diff --git a/yt_dlp/extractor/daftsex.py b/yt_dlp/extractor/daftsex.py
new file mode 100644
index 000000000..03672b35d
--- /dev/null
+++ b/yt_dlp/extractor/daftsex.py
@@ -0,0 +1,79 @@
+# coding: utf-8
+from __future__ import unicode_literals
+
+from .common import InfoExtractor
+from ..compat import compat_b64decode
+from ..utils import (
+    get_elements_by_class,
+    int_or_none,
+    js_to_json,
+    parse_count,
+    parse_duration,
+    try_get,
+)
+
+
+class DaftsexIE(InfoExtractor):
+    _VALID_URL = r'https?://(?:www\.)?daftsex\.com/watch/(?P<id>-?\d+_\d+)'
+    _TESTS = [{
+        'url': 'https://daftsex.com/watch/-156601359_456242791',
+        'info_dict': {
+            'id': '-156601359_456242791',
+            'ext': 'mp4',
+            'title': 'Skye Blue - Dinner And A Show',
+        },
+    }]
+
+    def _real_extract(self, url):
+        video_id = self._match_id(url)
+        webpage = self._download_webpage(url, video_id)
+        title = get_elements_by_class('heading', webpage)[-1]
+        duration = parse_duration(self._search_regex(
+            r'Duration: ((?:[0-9]{2}:){0,2}[0-9]{2})',
+            webpage, 'duration', fatal=False))
+        views = parse_count(self._search_regex(
+            r'Views: ([0-9 ]+)',
+            webpage, 'views', fatal=False))
+
+        player_hash = self._search_regex(
+            r'DaxabPlayer\.Init\({[\s\S]*hash:\s*"([0-9a-zA-Z_\-]+)"[\s\S]*}',
+            webpage, 'player hash')
+        player_color = self._search_regex(
+            r'DaxabPlayer\.Init\({[\s\S]*color:\s*"([0-9a-z]+)"[\s\S]*}',
+            webpage, 'player color', fatal=False) or ''
+
+        embed_page = self._download_webpage(
+            'https://daxab.com/player/%s?color=%s' % (player_hash, player_color),
+            video_id, headers={'Referer': url})
+        video_params = self._parse_json(
+            self._search_regex(
+                r'window\.globParams\s*=\s*({[\S\s]+})\s*;\s*<\/script>',
+                embed_page, 'video parameters'),
+            video_id, transform_source=js_to_json)
+
+        server_domain = 'https://%s' % compat_b64decode(video_params['server'][::-1]).decode('utf-8')
+        formats = []
+        for format_id, format_data in video_params['video']['cdn_files'].items():
+            ext, height = format_id.split('_')
+            extra_quality_data = format_data.split('.')[-1]
+            url = f'{server_domain}/videos/{video_id.replace("_", "/")}/{height}.mp4?extra={extra_quality_data}'
+            formats.append({
+                'format_id': format_id,
+                'url': url,
+                'height': int_or_none(height),
+                'ext': ext,
+            })
+        self._sort_formats(formats)
+
+        thumbnail = try_get(video_params,
+                            lambda vi: 'https:' + compat_b64decode(vi['video']['thumb']).decode('utf-8'))
+
+        return {
+            'id': video_id,
+            'title': title,
+            'formats': formats,
+            'duration': duration,
+            'thumbnail': thumbnail,
+            'view_count': views,
+            'age_limit': 18,
+        }
diff --git a/yt_dlp/extractor/extractors.py b/yt_dlp/extractor/extractors.py
index 194fe4be3..6e2ad3feb 100644
--- a/yt_dlp/extractor/extractors.py
+++ b/yt_dlp/extractor/extractors.py
@@ -321,6 +321,7 @@
     CuriosityStreamSeriesIE,
 )
 from .cwtv import CWTVIE
+from .daftsex import DaftsexIE
 from .dailymail import DailyMailIE
 from .dailymotion import (
     DailymotionIE,
@@ -828,7 +829,10 @@
 )
 from .mit import TechTVMITIE, OCWMITIE
 from .mitele import MiTeleIE
-from .mixch import MixchIE
+from .mixch import (
+    MixchIE,
+    MixchArchiveIE,
+)
 from .mixcloud import (
     MixcloudIE,
     MixcloudUserIE,
@@ -1518,7 +1522,12 @@
 from .teamcoco import TeamcocoIE
 from .teamtreehouse import TeamTreeHouseIE
 from .techtalks import TechTalksIE
-from .ted import TEDIE
+from .ted import (
+    TedEmbedIE,
+    TedPlaylistIE,
+    TedSeriesIE,
+    TedTalkIE,
+)
 from .tele5 import Tele5IE
 from .tele13 import Tele13IE
 from .telebruxelles import TeleBruxellesIE
@@ -1965,6 +1974,7 @@
     YoutubeFavouritesIE,
     YoutubeHistoryIE,
     YoutubeTabIE,
+    YoutubeLivestreamEmbedIE,
     YoutubePlaylistIE,
     YoutubeRecommendedIE,
     YoutubeSearchDateIE,
diff --git a/yt_dlp/extractor/generic.py b/yt_dlp/extractor/generic.py
index 864977994..9b62ebfed 100644
--- a/yt_dlp/extractor/generic.py
+++ b/yt_dlp/extractor/generic.py
@@ -115,6 +115,7 @@
 from .vshare import VShareIE
 from .mediasite import MediasiteIE
 from .springboardplatform import SpringboardPlatformIE
+from .ted import TedEmbedIE
 from .yapfiles import YapFilesIE
 from .vice import ViceIE
 from .xfileshare import XFileShareIE
@@ -1885,8 +1886,42 @@ class GenericIE(InfoExtractor):
                 'timestamp': 1641885019,
                 'upload_date': '20220111',
                 'duration': 460000,
+                'thumbnail': 'https://i3thumbs.glomex.com/dC1idjJwdndiMjRzeGwvMjAyMi8wMS8xMS8wNy8xMF8zNV82MWRkMmQ2YmU5ZTgyLmpwZw==/profile:player-960x540',
             },
         },
+        {
+            # megatvcom:embed
+            'url': 'https://www.in.gr/2021/12/18/greece/apokalypsi-mega-poios-parelave-tin-ereyna-tsiodra-ek-merous-tis-kyvernisis-o-prothypourgos-telika-gnorize/',
+            'info_dict': {
+                'id': 'apokalypsi-mega-poios-parelave-tin-ereyna-tsiodra-ek-merous-tis-kyvernisis-o-prothypourgos-telika-gnorize',
+                'title': 'md5:5e569cf996ec111057c2764ec272848f',
+            },
+            'playlist': [{
+                'md5': '1afa26064ff00ccb91617957dbc73dc1',
+                'info_dict': {
+                    'ext': 'mp4',
+                    'id': '564916',
+                    'display_id': 'md5:6cdf22d3a2e7bacb274b7295089a1770',
+                    'title': 'md5:33b9dd39584685b62873043670eb52a6',
+                    'description': 'md5:c1db7310f390518ac36dd69d947ef1a1',
+                    'timestamp': 1639753145,
+                    'upload_date': '20211217',
+                    'thumbnail': 'https://www.megatv.com/wp-content/uploads/2021/12/prezerakos-1024x597.jpg',
+                },
+            }, {
+                'md5': '4a1c220695f1ef865a8b7966a53e2474',
+                'info_dict': {
+                    'ext': 'mp4',
+                    'id': '564905',
+                    'display_id': 'md5:ead15695e485e649aed2b81ebd699b88',
+                    'title': 'md5:2b71fd54249a3ca34609fe39ae31c47b',
+                    'description': 'md5:c42e12f638d0a97d6de4508e2c4df982',
+                    'timestamp': 1639753047,
+                    'upload_date': '20211217',
+                    'thumbnail': 'https://www.megatv.com/wp-content/uploads/2021/12/tsiodras-mitsotakis-1024x545.jpg',
+                },
+            }]
+        },
         {
             # ThePlatform embedded with whitespaces in URLs
             'url': 'http://www.golfchannel.com/topics/shows/golftalkcentral.htm',
@@ -3140,10 +3175,9 @@ def _real_extract(self, url):
             return self.url_result(mobj.group('url'), 'Tvigle')
 
         # Look for embedded TED player
-        mobj = re.search(
-            r'<iframe[^>]+?src=(["\'])(?P<url>https?://embed(?:-ssl)?\.ted\.com/.+?)\1', webpage)
-        if mobj is not None:
-            return self.url_result(mobj.group('url'), 'TED')
+        ted_urls = TedEmbedIE._extract_urls(webpage)
+        if ted_urls:
+            return self.playlist_from_matches(ted_urls, video_id, video_title, ie=TedEmbedIE.ie_key())
 
         # Look for embedded Ustream videos
         ustream_url = UstreamIE._extract_url(webpage)
diff --git a/yt_dlp/extractor/mixch.py b/yt_dlp/extractor/mixch.py
index a99ddd172..31f450dfa 100644
--- a/yt_dlp/extractor/mixch.py
+++ b/yt_dlp/extractor/mixch.py
@@ -11,7 +11,7 @@ class MixchIE(InfoExtractor):
     IE_NAME = 'mixch'
     _VALID_URL = r'https?://(?:www\.)?mixch\.tv/u/(?P<id>\d+)'
 
-    TESTS = [{
+    _TESTS = [{
         'url': 'https://mixch.tv/u/16236849/live',
         'skip': 'don\'t know if this live persists',
         'info_dict': {
@@ -53,3 +53,33 @@ def _real_extract(self, url):
             }],
             'is_live': True,
         }
+
+
+class MixchArchiveIE(InfoExtractor):
+    IE_NAME = 'mixch:archive'
+    _VALID_URL = r'https?://(?:www\.)?mixch\.tv/archive/(?P<id>\d+)'
+
+    _TESTS = [{
+        'url': 'https://mixch.tv/archive/421',
+        'skip': 'paid video, no DRM. expires at Jan 23',
+        'info_dict': {
+            'id': '421',
+            'title': '96NEKO SHOW TIME',
+        }
+    }]
+
+    def _real_extract(self, url):
+        video_id = self._match_id(url)
+        webpage = self._download_webpage(url, video_id)
+
+        html5_videos = self._parse_html5_media_entries(
+            url, webpage.replace('video-js', 'video'), video_id, 'hls')
+        if not html5_videos:
+            self.raise_login_required(method='cookies')
+        infodict = html5_videos[0]
+        infodict.update({
+            'id': video_id,
+            'title': self._html_search_regex(r'class="archive-title">(.+?)</', webpage, 'title')
+        })
+
+        return infodict
diff --git a/yt_dlp/extractor/pbs.py b/yt_dlp/extractor/pbs.py
index ffaa6bf92..e48a2b8e0 100644
--- a/yt_dlp/extractor/pbs.py
+++ b/yt_dlp/extractor/pbs.py
@@ -545,7 +545,7 @@ def _real_extract(self, url):
                 for vid_id in video_id]
             return self.playlist_result(entries, display_id)
 
-        info = None
+        info = {}
         redirects = []
         redirect_urls = set()
 
@@ -660,6 +660,9 @@ def extract_redirect_urls(info):
                     'protocol': 'http',
                 })
                 formats.append(f)
+        for f in formats:
+            if (f.get('format_note') or '').endswith(' AD'):  # Audio description
+                f['language_preference'] = -10
         self._sort_formats(formats)
 
         rating_str = info.get('rating')
diff --git a/yt_dlp/extractor/ted.py b/yt_dlp/extractor/ted.py
index f09f1a3f9..b5c7e35ac 100644
--- a/yt_dlp/extractor/ted.py
+++ b/yt_dlp/extractor/ted.py
@@ -1,274 +1,105 @@
-from __future__ import unicode_literals
-
-import json
+import itertools
 import re
 
 from .common import InfoExtractor
 
-from ..compat import (
-    compat_str,
-    compat_urlparse
-)
 from ..utils import (
-    extract_attributes,
-    float_or_none,
     int_or_none,
+    str_to_int,
     try_get,
     url_or_none,
+    unified_strdate,
+    parse_duration,
 )
 
 
-class TEDIE(InfoExtractor):
-    IE_NAME = 'ted'
-    _VALID_URL = r'''(?x)
-        (?P<proto>https?://)
-        (?P<type>www|embed(?:-ssl)?)(?P<urlmain>\.ted\.com/
-        (
-            (?P<type_playlist>playlists(?:/(?P<playlist_id>\d+))?) # We have a playlist
-            |
-            ((?P<type_talk>talks)) # We have a simple talk
-            |
-            (?P<type_watch>watch)/[^/]+/[^/]+
-        )
-        (/lang/(.*?))? # The url may contain the language
-        /(?P<name>[\w-]+) # Here goes the name and then ".html"
-        .*)$
-        '''
+class TedBaseIE(InfoExtractor):
+    _VALID_URL_BASE = r'https?://www\.ted\.com/(?:{type})(?:/lang/[^/#?]+)?/(?P<id>[\w-]+)'
+
+    def _parse_playlist(self, playlist):
+        for entry in try_get(playlist, lambda x: x['videos']['nodes'], list):
+            if entry.get('__typename') == 'Video' and entry.get('canonicalUrl'):
+                yield self.url_result(entry['canonicalUrl'], TedTalkIE.ie_key())
+
+
+class TedTalkIE(TedBaseIE):
+    _VALID_URL = TedBaseIE._VALID_URL_BASE.format(type='talks')
     _TESTS = [{
-        'url': 'http://www.ted.com/talks/dan_dennett_on_our_consciousness.html',
-        'md5': 'b0ce2b05ca215042124fbc9e3886493a',
-        'info_dict': {
-            'id': '102',
-            'ext': 'mp4',
-            'title': 'The illusion of consciousness',
-            'description': ('Philosopher Dan Dennett makes a compelling '
-                            'argument that not only don\'t we understand our own '
-                            'consciousness, but that half the time our brains are '
-                            'actively fooling us.'),
-            'uploader': 'Dan Dennett',
-            'width': 853,
-            'duration': 1308,
-            'view_count': int,
-            'comment_count': int,
-            'tags': list,
-        },
-        'params': {
-            'skip_download': True,
-        },
-    }, {
-        # missing HTTP bitrates
-        'url': 'https://www.ted.com/talks/vishal_sikka_the_beauty_and_power_of_algorithms',
-        'info_dict': {
-            'id': '6069',
-            'ext': 'mp4',
-            'title': 'The beauty and power of algorithms',
-            'thumbnail': r're:^https?://.+\.jpg',
-            'description': 'md5:734e352710fb00d840ab87ae31aaf688',
-            'uploader': 'Vishal Sikka',
-        },
-        'params': {
-            'skip_download': True,
-        },
-    }, {
-        'url': 'http://www.ted.com/talks/gabby_giffords_and_mark_kelly_be_passionate_be_courageous_be_your_best',
-        'md5': 'e6b9617c01a7970ceac8bb2c92c346c0',
-        'info_dict': {
-            'id': '1972',
-            'ext': 'mp4',
-            'title': 'Be passionate. Be courageous. Be your best.',
-            'uploader': 'Gabby Giffords and Mark Kelly',
-            'description': 'md5:5174aed4d0f16021b704120360f72b92',
-            'duration': 1128,
-        },
-        'params': {
-            'skip_download': True,
-        },
-    }, {
-        'url': 'http://www.ted.com/playlists/who_are_the_hackers',
-        'info_dict': {
-            'id': '10',
-            'title': 'Who are the hackers?',
-            'description': 'md5:49a0dbe8fb76d81a0e64b4a80af7f15a'
-        },
-        'playlist_mincount': 6,
-    }, {
-        # contains a youtube video
-        'url': 'https://www.ted.com/talks/douglas_adams_parrots_the_universe_and_everything',
-        'add_ie': ['Youtube'],
-        'info_dict': {
-            'id': '_ZG8HBuDjgc',
-            'ext': 'webm',
-            'title': 'Douglas Adams: Parrots the Universe and Everything',
-            'description': 'md5:01ad1e199c49ac640cb1196c0e9016af',
-            'uploader': 'University of California Television (UCTV)',
-            'uploader_id': 'UCtelevision',
-            'upload_date': '20080522',
-        },
-        'params': {
-            'skip_download': True,
-        },
-    }, {
-        # no nativeDownloads
-        'url': 'https://www.ted.com/talks/tom_thum_the_orchestra_in_my_mouth',
+        'url': 'https://www.ted.com/talks/candace_parker_how_to_break_down_barriers_and_not_accept_limits',
+        'md5': '47e82c666d9c3261d4fe74748a90aada',
         'info_dict': {
-            'id': '1792',
+            'id': '86532',
             'ext': 'mp4',
-            'title': 'The orchestra in my mouth',
-            'description': 'md5:5d1d78650e2f8dfcbb8ebee2951ac29a',
-            'uploader': 'Tom Thum',
+            'title': 'How to break down barriers and not accept limits',
+            'description': 'md5:000707cece219d1e165b11550d612331',
             'view_count': int,
-            'comment_count': int,
-            'tags': list,
+            'tags': ['personal growth', 'equality', 'activism', 'motivation', 'social change', 'sports'],
+            'uploader': 'Candace Parker',
+            'duration': 676.0,
+            'upload_date': '20220114',
+            'release_date': '20211201',
+            'thumbnail': r're:http.*\.jpg',
         },
-        'params': {
-            'skip_download': True,
-        },
-    }, {
-        # with own formats and private Youtube external
-        'url': 'https://www.ted.com/talks/spencer_wells_a_family_tree_for_humanity',
-        'only_matching': True,
     }]
 
-    _NATIVE_FORMATS = {
-        'low': {'width': 320, 'height': 180},
-        'medium': {'width': 512, 'height': 288},
-        'high': {'width': 854, 'height': 480},
-    }
-
-    def _extract_info(self, webpage):
-        info_json = self._search_regex(
-            r'(?s)q\(\s*"\w+.init"\s*,\s*({.+?})\)\s*</script>',
-            webpage, 'info json')
-        return json.loads(info_json)
-
     def _real_extract(self, url):
-        m = re.match(self._VALID_URL, url, re.VERBOSE)
-        if m.group('type').startswith('embed'):
-            desktop_url = m.group('proto') + 'www' + m.group('urlmain')
-            return self.url_result(desktop_url, 'TED')
-        name = m.group('name')
-        if m.group('type_talk'):
-            return self._talk_info(url, name)
-        elif m.group('type_watch'):
-            return self._watch_info(url, name)
-        else:
-            return self._playlist_videos_info(url, name)
-
-    def _playlist_videos_info(self, url, name):
-        '''Returns the videos of the playlist'''
-
-        webpage = self._download_webpage(url, name,
-                                         'Downloading playlist webpage')
-
-        playlist_entries = []
-        for entry in re.findall(r'(?s)<[^>]+data-ga-context=["\']playlist["\'][^>]*>', webpage):
-            attrs = extract_attributes(entry)
-            entry_url = compat_urlparse.urljoin(url, attrs['href'])
-            playlist_entries.append(self.url_result(entry_url, self.ie_key()))
-
-        final_url = self._og_search_url(webpage, fatal=False)
-        playlist_id = (
-            re.match(self._VALID_URL, final_url).group('playlist_id')
-            if final_url else None)
-
-        return self.playlist_result(
-            playlist_entries, playlist_id=playlist_id,
-            playlist_title=self._og_search_title(webpage, fatal=False),
-            playlist_description=self._og_search_description(webpage))
-
-    def _talk_info(self, url, video_name):
-        webpage = self._download_webpage(url, video_name)
-
-        info = self._extract_info(webpage)
-
-        data = try_get(info, lambda x: x['__INITIAL_DATA__'], dict) or info
-        talk_info = data['talks'][0]
-
-        title = talk_info['title'].strip()
-
-        downloads = talk_info.get('downloads') or {}
-        native_downloads = downloads.get('nativeDownloads') or talk_info.get('nativeDownloads') or {}
-
-        formats = [{
-            'url': format_url,
-            'format_id': format_id,
-        } for (format_id, format_url) in native_downloads.items() if format_url is not None]
-
-        subtitled_downloads = downloads.get('subtitledDownloads') or {}
-        for lang, subtitled_download in subtitled_downloads.items():
-            for q in self._NATIVE_FORMATS:
-                q_url = subtitled_download.get(q)
-                if not q_url:
-                    continue
-                formats.append({
-                    'url': q_url,
-                    'format_id': '%s-%s' % (q, lang),
-                    'language': lang,
-                })
-
-        if formats:
-            for f in formats:
-                finfo = self._NATIVE_FORMATS.get(f['format_id'].split('-')[0])
-                if finfo:
-                    f.update(finfo)
-
-        player_talk = talk_info['player_talks'][0]
-
-        resources_ = player_talk.get('resources') or talk_info.get('resources')
+        display_id = self._match_id(url)
+        webpage = self._download_webpage(url, display_id)
+        talk_info = self._search_nextjs_data(webpage, display_id)['props']['pageProps']['videoData']
+        video_id = talk_info['id']
+        playerData = self._parse_json(talk_info.get('playerData'), video_id)
 
         http_url = None
-        for format_id, resources in resources_.items():
+        formats, subtitles = [], {}
+        for format_id, resources in (playerData.get('resources') or {}).items():
             if format_id == 'hls':
-                if not isinstance(resources, dict):
-                    continue
-                stream_url = url_or_none(resources.get('stream'))
+                stream_url = url_or_none(try_get(resources, lambda x: x['stream']))
                 if not stream_url:
                     continue
-                formats.extend(self._extract_m3u8_formats(
-                    stream_url, video_name, 'mp4', m3u8_id=format_id,
-                    fatal=False))
-            else:
-                if not isinstance(resources, list):
-                    continue
-                if format_id == 'h264':
-                    for resource in resources:
-                        h264_url = resource.get('file')
-                        if not h264_url:
-                            continue
-                        bitrate = int_or_none(resource.get('bitrate'))
-                        formats.append({
-                            'url': h264_url,
-                            'format_id': '%s-%sk' % (format_id, bitrate),
-                            'tbr': bitrate,
-                        })
-                        if re.search(r'\d+k', h264_url):
-                            http_url = h264_url
-                elif format_id == 'rtmp':
-                    streamer = talk_info.get('streamer')
-                    if not streamer:
+                m3u8_formats, m3u8_subs = self._extract_m3u8_formats_and_subtitles(
+                    stream_url, video_id, 'mp4', m3u8_id=format_id, fatal=False)
+                formats.extend(m3u8_formats)
+                subtitles = self._merge_subtitles(subtitles, m3u8_subs)
+                continue
+
+            if not isinstance(resources, list):
+                continue
+            if format_id == 'h264':
+                for resource in resources:
+                    h264_url = resource.get('file')
+                    if not h264_url:
                         continue
-                    for resource in resources:
-                        formats.append({
-                            'format_id': '%s-%s' % (format_id, resource.get('name')),
-                            'url': streamer,
-                            'play_path': resource['file'],
-                            'ext': 'flv',
-                            'width': int_or_none(resource.get('width')),
-                            'height': int_or_none(resource.get('height')),
-                            'tbr': int_or_none(resource.get('bitrate')),
-                        })
+                    bitrate = int_or_none(resource.get('bitrate'))
+                    formats.append({
+                        'url': h264_url,
+                        'format_id': '%s-%sk' % (format_id, bitrate),
+                        'tbr': bitrate,
+                    })
+                    if re.search(r'\d+k', h264_url):
+                        http_url = h264_url
+            elif format_id == 'rtmp':
+                streamer = talk_info.get('streamer')
+                if not streamer:
+                    continue
+                formats.extend({
+                    'format_id': '%s-%s' % (format_id, resource.get('name')),
+                    'url': streamer,
+                    'play_path': resource['file'],
+                    'ext': 'flv',
+                    'width': int_or_none(resource.get('width')),
+                    'height': int_or_none(resource.get('height')),
+                    'tbr': int_or_none(resource.get('bitrate')),
+                } for resource in resources if resource.get('file'))
 
-        m3u8_formats = list(filter(
-            lambda f: f.get('protocol') == 'm3u8' and f.get('vcodec') != 'none',
-            formats))
         if http_url:
+            m3u8_formats = [f for f in formats if f.get('protocol') == 'm3u8' and f.get('vcodec') != 'none']
             for m3u8_format in m3u8_formats:
                 bitrate = self._search_regex(r'(\d+k)', m3u8_format['url'], 'bitrate', default=None)
                 if not bitrate:
                     continue
                 bitrate_url = re.sub(r'\d+k', bitrate, http_url)
                 if not self._is_valid_url(
-                        bitrate_url, video_name, '%s bitrate' % bitrate):
+                        bitrate_url, video_id, '%s bitrate' % bitrate):
                     continue
                 f = m3u8_format.copy()
                 f.update({
@@ -289,79 +120,123 @@ def _talk_info(self, url, video_name):
             })
 
         if not formats:
-            external = player_talk.get('external')
-            if isinstance(external, dict):
-                service = external.get('service')
-                if isinstance(service, compat_str):
-                    ext_url = None
-                    if service.lower() == 'youtube':
-                        ext_url = external.get('code')
-                    return self.url_result(ext_url or external['uri'])
+            external = playerData.get('external') or {}
+            service = external.get('service') or ''
+            ext_url = external.get('code') if service.lower() == 'youtube' else None
+            return self.url_result(ext_url or external['uri'])
 
         self._sort_formats(formats)
 
-        video_id = compat_str(talk_info['id'])
+        thumbnail = playerData.get('thumb') or self._og_search_property('image', webpage)
+        if thumbnail:
+            # trim thumbnail resize parameters
+            thumbnail = thumbnail.split('?')[0]
 
         return {
             'id': video_id,
-            'title': title,
-            'uploader': player_talk.get('speaker') or talk_info.get('speaker'),
-            'thumbnail': player_talk.get('thumb') or talk_info.get('thumb'),
-            'description': self._og_search_description(webpage),
-            'subtitles': self._get_subtitles(video_id, talk_info),
+            'title': talk_info.get('title') or self._og_search_title(webpage),
+            'uploader': talk_info.get('presenterDisplayName'),
+            'thumbnail': thumbnail,
+            'description': talk_info.get('description') or self._og_search_description(webpage),
+            'subtitles': subtitles,
             'formats': formats,
-            'duration': float_or_none(talk_info.get('duration')),
-            'view_count': int_or_none(data.get('viewed_count')),
-            'comment_count': int_or_none(
-                try_get(data, lambda x: x['comments']['count'])),
-            'tags': try_get(talk_info, lambda x: x['tags'], list),
+            'duration': talk_info.get('duration') or parse_duration(self._og_search_property('video:duration', webpage)),
+            'view_count': str_to_int(talk_info.get('viewedCount')),
+            'upload_date': unified_strdate(talk_info.get('publishedAt')),
+            'release_date': unified_strdate(talk_info.get('recordedOn')),
+            'tags': try_get(playerData, lambda x: x['targeting']['tag'].split(',')),
         }
 
-    def _get_subtitles(self, video_id, talk_info):
-        sub_lang_list = {}
-        for language in try_get(
-                talk_info,
-                (lambda x: x['downloads']['languages'],
-                 lambda x: x['languages']), list):
-            lang_code = language.get('languageCode') or language.get('ianaCode')
-            if not lang_code:
-                continue
-            sub_lang_list[lang_code] = [
-                {
-                    'url': 'http://www.ted.com/talks/subtitles/id/%s/lang/%s/format/%s' % (video_id, lang_code, ext),
-                    'ext': ext,
-                }
-                for ext in ['ted', 'srt']
-            ]
-        return sub_lang_list
 
-    def _watch_info(self, url, name):
-        webpage = self._download_webpage(url, name)
+class TedSeriesIE(TedBaseIE):
+    _VALID_URL = fr'{TedBaseIE._VALID_URL_BASE.format(type=r"series")}(?:#season_(?P<season>\d+))?'
+    _TESTS = [{
+        'url': 'https://www.ted.com/series/small_thing_big_idea',
+        'info_dict': {
+            'id': '3',
+            'title': 'Small Thing Big Idea',
+            'series': 'Small Thing Big Idea',
+            'description': 'md5:6869ca52cec661aef72b3e9f7441c55c'
+        },
+        'playlist_mincount': 16,
+    }, {
+        'url': 'https://www.ted.com/series/the_way_we_work#season_2',
+        'info_dict': {
+            'id': '8_2',
+            'title': 'The Way We Work Season 2',
+            'series': 'The Way We Work',
+            'description': 'md5:59469256e533e1a48c4aa926a382234c',
+            'season_number': 2
+        },
+        'playlist_mincount': 8,
+    }]
 
-        config_json = self._html_search_regex(
-            r'"pages\.jwplayer"\s*,\s*({.+?})\s*\)\s*</script>',
-            webpage, 'config', default=None)
-        if not config_json:
-            embed_url = self._search_regex(
-                r"<iframe[^>]+class='pages-video-embed__video__object'[^>]+src='([^']+)'", webpage, 'embed url')
-            return self.url_result(self._proto_relative_url(embed_url))
-        config = json.loads(config_json)['config']
-        video_url = config['video']['url']
-        thumbnail = config.get('image', {}).get('url')
+    def _real_extract(self, url):
+        display_id, season = self._match_valid_url(url).group('id', 'season')
+        webpage = self._download_webpage(url, display_id, 'Downloading series webpage')
+        info = self._search_nextjs_data(webpage, display_id)['props']['pageProps']
 
-        title = self._html_search_regex(
-            r"(?s)<h1(?:\s+class='[^']+')?>(.+?)</h1>", webpage, 'title')
-        description = self._html_search_regex(
-            [
-                r'(?s)<h4 class="[^"]+" id="h3--about-this-talk">.*?</h4>(.*?)</div>',
-                r'(?s)<p><strong>About this talk:</strong>\s+(.*?)</p>',
-            ],
-            webpage, 'description', fatal=False)
+        entries = itertools.chain.from_iterable(
+            self._parse_playlist(s) for s in info['seasons'] if season in [None, s.get('seasonNumber')])
 
-        return {
-            'id': name,
-            'url': video_url,
-            'title': title,
-            'thumbnail': thumbnail,
-            'description': description,
-        }
+        series_id = try_get(info, lambda x: x['series']['id'])
+        series_name = try_get(info, lambda x: x['series']['name']) or self._og_search_title(webpage, fatal=False)
+
+        return self.playlist_result(
+            entries,
+            f'{series_id}_{season}' if season and series_id else series_id,
+            f'{series_name} Season {season}' if season else series_name,
+            self._og_search_description(webpage),
+            series=series_name, season_number=int_or_none(season))
+
+
+class TedPlaylistIE(TedBaseIE):
+    _VALID_URL = TedBaseIE._VALID_URL_BASE.format(type=r'playlists(?:/\d+)?')
+    _TESTS = [{
+        'url': 'https://www.ted.com/playlists/171/the_most_popular_talks_of_all',
+        'info_dict': {
+            'id': '171',
+            'title': 'The most popular talks of all time',
+            'description': 'md5:d2f22831dc86c7040e733a3cb3993d78'
+        },
+        'playlist_mincount': 25,
+    }]
+
+    def _real_extract(self, url):
+        display_id = self._match_id(url)
+        webpage = self._download_webpage(url, display_id)
+        playlist = self._search_nextjs_data(webpage, display_id)['props']['pageProps']['playlist']
+
+        return self.playlist_result(
+            self._parse_playlist(playlist), playlist.get('id'),
+            playlist.get('title') or self._og_search_title(webpage, default='').replace(' | TED Talks', '') or None,
+            self._og_search_description(webpage))
+
+
+class TedEmbedIE(InfoExtractor):
+    _VALID_URL = r'https?://embed(?:-ssl)?\.ted\.com/'
+
+    _TESTS = [{
+        'url': 'https://embed.ted.com/talks/janet_stovall_how_to_get_serious_about_diversity_and_inclusion_in_the_workplace',
+        'info_dict': {
+            'id': '21802',
+            'ext': 'mp4',
+            'title': 'How to get serious about diversity and inclusion in the workplace',
+            'description': 'md5:0978aafe396e05341f8ecc795d22189d',
+            'view_count': int,
+            'tags': list,
+            'uploader': 'Janet Stovall',
+            'duration': 664.0,
+            'upload_date': '20180822',
+            'release_date': '20180719',
+            'thumbnail': r're:http.*\.jpg',
+        },
+    }]
+
+    @classmethod
+    def _extract_urls(cls, webpage):
+        return [mobj.group('url') for mobj in re.finditer(
+            fr'<iframe[^>]+?src=(["\'])(?P<url>{cls._VALID_URL}.+?)\1', webpage)]
+
+    def _real_extract(self, url):
+        return self.url_result(re.sub(r'://embed(-ssl)?', '://www', url), TedTalkIE.ie_key())
diff --git a/yt_dlp/extractor/thisoldhouse.py b/yt_dlp/extractor/thisoldhouse.py
index a3d9b4017..35c69a988 100644
--- a/yt_dlp/extractor/thisoldhouse.py
+++ b/yt_dlp/extractor/thisoldhouse.py
@@ -2,6 +2,7 @@
 from __future__ import unicode_literals
 
 from .common import InfoExtractor
+from ..utils import HEADRequest
 
 
 class ThisOldHouseIE(InfoExtractor):
@@ -15,6 +16,11 @@ class ThisOldHouseIE(InfoExtractor):
             'description': 'In the workshop, Tom Silva and Kevin O\'Connor build a storage bench for an entryway.',
             'timestamp': 1442548800,
             'upload_date': '20150918',
+            'duration': 674,
+            'view_count': int,
+            'average_rating': 0,
+            'thumbnail': r're:^https?://.*\.jpg\?\d+$',
+            'display_id': 'how-to-build-a-storage-bench',
         },
         'params': {
             'skip_download': True,
@@ -41,7 +47,12 @@ class ThisOldHouseIE(InfoExtractor):
     def _real_extract(self, url):
         display_id = self._match_id(url)
         webpage = self._download_webpage(url, display_id)
-        video_id = self._search_regex(
-            r'<iframe[^>]+src=[\'"](?:https?:)?//(?:www\.)?thisoldhouse\.(?:chorus\.build|com)/videos/zype/([0-9a-f]{24})',
-            webpage, 'video id')
+        if 'To Unlock This content' in webpage:
+            self.raise_login_required(method='cookies')
+        video_url = self._search_regex(
+            r'<iframe[^>]+src=[\'"]((?:https?:)?//(?:www\.)?thisoldhouse\.(?:chorus\.build|com)/videos/zype/([0-9a-f]{24})[^\'"]*)[\'"]',
+            webpage, 'video url')
+        if 'subscription_required=true' in video_url:
+            return self.url_result(self._request_webpage(HEADRequest(video_url), display_id).geturl(), 'Zype', display_id)
+        video_id = self._search_regex(r'(?:https?:)?//(?:www\.)?thisoldhouse\.(?:chorus\.build|com)/videos/zype/([0-9a-f]{24})', video_url, 'video id')
         return self.url_result(self._ZYPE_TMPL % video_id, 'Zype', video_id)
diff --git a/yt_dlp/extractor/tver.py b/yt_dlp/extractor/tver.py
index 943b3ebdd..b8ac41483 100644
--- a/yt_dlp/extractor/tver.py
+++ b/yt_dlp/extractor/tver.py
@@ -5,10 +5,11 @@
 from .common import InfoExtractor
 from ..compat import compat_str
 from ..utils import (
+    ExtractorError,
     int_or_none,
     remove_start,
     smuggle_url,
-    try_get,
+    traverse_obj,
 )
 
 
@@ -38,13 +39,18 @@ def _real_initialize(self):
 
     def _real_extract(self, url):
         path, video_id = self._match_valid_url(url).groups()
-        main = self._download_json(
+        api_response = self._download_json(
             'https://api.tver.jp/v4/' + path, video_id,
-            query={'token': self._TOKEN})['main']
-        p_id = main['publisher_id']
-        service = remove_start(main['service'], 'ts_')
+            query={'token': self._TOKEN})
+        p_id = traverse_obj(api_response, ('main', 'publisher_id'))
+        if not p_id:
+            error_msg, expected = traverse_obj(api_response, ('episode', 0, 'textbar', 0, ('text', 'longer')), get_all=False), True
+            if not error_msg:
+                error_msg, expected = 'Failed to extract publisher ID', False
+            raise ExtractorError(error_msg, expected=expected)
+        service = remove_start(traverse_obj(api_response, ('main', 'service')), 'ts_')
 
-        r_id = main['reference_id']
+        r_id = traverse_obj(api_response, ('main', 'reference_id'))
         if service not in ('tx', 'russia2018', 'sebare2018live', 'gorin'):
             r_id = 'ref:' + r_id
         bc_url = smuggle_url(
@@ -53,8 +59,8 @@ def _real_extract(self, url):
 
         return {
             '_type': 'url_transparent',
-            'description': try_get(main, lambda x: x['note'][0]['text'], compat_str),
-            'episode_number': int_or_none(try_get(main, lambda x: x['ext']['episode_number'])),
+            'description': traverse_obj(api_response, ('main', 'note', 0, 'text'), expected_type=compat_str),
+            'episode_number': int_or_none(traverse_obj(api_response, ('main', 'ext', 'episode_number'), expected_type=compat_str)),
             'url': bc_url,
             'ie_key': 'BrightcoveNew',
         }
diff --git a/yt_dlp/extractor/youtube.py b/yt_dlp/extractor/youtube.py
index 244115f48..d8b4ad258 100644
--- a/yt_dlp/extractor/youtube.py
+++ b/yt_dlp/extractor/youtube.py
@@ -839,7 +839,7 @@ def _extract_video(self, renderer):
             'uploader': uploader,
             'channel_id': channel_id,
             'thumbnails': thumbnails,
-            'upload_date': strftime_or_none(timestamp, '%Y%m%d'),
+            #  'upload_date': strftime_or_none(timestamp, '%Y%m%d'),
             'live_status': ('is_upcoming' if scheduled_timestamp is not None
                             else 'was_live' if 'streamed' in time_text.lower()
                             else 'is_live' if overlay_style is not None and overlay_style == 'LIVE' or 'live now' in badges
@@ -864,7 +864,7 @@ class YoutubeIE(YoutubeBaseInfoExtractor):
                             youtube\.googleapis\.com)/                        # the various hostnames, with wildcard subdomains
                          (?:.*?\#/)?                                          # handle anchor (#/) redirect urls
                          (?:                                                  # the various things that can precede the ID:
-                             (?:(?:v|embed|e|shorts)/(?!videoseries))         # v/ or embed/ or e/ or shorts/
+                             (?:(?:v|embed|e|shorts)/(?!videoseries|live_stream))  # v/ or embed/ or e/ or shorts/
                              |(?:                                             # or the v= param in all its forms
                                  (?:(?:watch|movie)(?:_popup)?(?:\.php)?/?)?  # preceding watch(_popup|.php) or nothing (like /?v=xxxx)
                                  (?:\?|\#!?)                                  # the params delimiter ? or # or #!
@@ -3058,9 +3058,9 @@ def process_manifest_format(f, proto, itag):
     def _extract_storyboard(self, player_responses, duration):
         spec = get_first(
             player_responses, ('storyboards', 'playerStoryboardSpecRenderer', 'spec'), default='').split('|')[::-1]
-        if not spec:
+        base_url = url_or_none(urljoin('https://i.ytimg.com/', spec.pop() or None))
+        if not base_url:
             return
-        base_url = spec.pop()
         L = len(spec) - 1
         for i, args in enumerate(spec):
             args = args.split('#')
@@ -5139,8 +5139,24 @@ def _real_extract(self, url):
             }), ie=YoutubeTabIE.ie_key(), video_id=playlist_id)
 
 
+class YoutubeLivestreamEmbedIE(InfoExtractor):
+    IE_DESC = 'YouTube livestream embeds'
+    _VALID_URL = r'https?://(?:\w+\.)?youtube\.com/embed/live_stream/?\?(?:[^#]+&)?channel=(?P<id>[^&#]+)'
+    _TESTS = [{
+        'url': 'https://www.youtube.com/embed/live_stream?channel=UC2_KI6RB__jGdlnK6dvFEZA',
+        'only_matching': True,
+    }]
+
+    def _real_extract(self, url):
+        channel_id = self._match_id(url)
+        return self.url_result(
+            f'https://www.youtube.com/channel/{channel_id}/live',
+            ie=YoutubeTabIE.ie_key(), video_id=channel_id)
+
+
 class YoutubeYtUserIE(InfoExtractor):
     IE_DESC = 'YouTube user videos; "ytuser:" prefix'
+    IE_NAME = 'youtube:user'
     _VALID_URL = r'ytuser:(?P<id>.+)'
     _TESTS = [{
         'url': 'ytuser:phihag',
diff --git a/yt_dlp/utils.py b/yt_dlp/utils.py
index b7e718028..7adfb1e74 100644
--- a/yt_dlp/utils.py
+++ b/yt_dlp/utils.py
@@ -4094,9 +4094,14 @@ def parse_duration(s):
         return None
 
     days, hours, mins, secs, ms = [None] * 5
-    m = re.match(r'(?:(?:(?:(?P<days>[0-9]+):)?(?P<hours>[0-9]+):)?(?P<mins>[0-9]+):)?(?P<secs>[0-9]+)(?P<ms>\.[0-9]+)?Z?$', s)
+    m = re.match(r'''(?x)
+            (?P<before_secs>
+                (?:(?:(?P<days>[0-9]+):)?(?P<hours>[0-9]+):)?(?P<mins>[0-9]+):)?
+            (?P<secs>(?(before_secs)[0-9]{1,2}|[0-9]+))
+            (?P<ms>[.:][0-9]+)?Z?$
+        ''', s)
     if m:
-        days, hours, mins, secs, ms = m.groups()
+        days, hours, mins, secs, ms = m.group('days', 'hours', 'mins', 'secs', 'ms')
     else:
         m = re.match(
             r'''(?ix)(?:P?
@@ -4141,7 +4146,7 @@ def parse_duration(s):
     if days:
         duration += float(days) * 24 * 60 * 60
     if ms:
-        duration += float(ms)
+        duration += float(ms.replace(':', '.'))
     return duration
 
 


Test patch:
diff --git a/test/test_subtitles.py b/test/test_subtitles.py
index 9b39dbd39..95e33e54a 100644
--- a/test/test_subtitles.py
+++ b/test/test_subtitles.py
@@ -13,7 +13,7 @@
 from yt_dlp.extractor import (
     YoutubeIE,
     DailymotionIE,
-    TEDIE,
+    TedTalkIE,
     VimeoIE,
     WallaIE,
     CeskaTelevizeIE,
@@ -141,7 +141,7 @@ def test_nosubtitles(self):
 @is_download_test
 class TestTedSubtitles(BaseTestSubtitles):
     url = 'http://www.ted.com/talks/dan_dennett_on_our_consciousness.html'
-    IE = TEDIE
+    IE = TedTalkIE
 
     def test_allsubtitles(self):
         self.DL.params['writesubtitles'] = True
diff --git a/test/test_utils.py b/test/test_utils.py
index 2c8f2c03e..11a13a691 100644
--- a/test/test_utils.py
+++ b/test/test_utils.py
@@ -643,6 +643,8 @@ def test_parse_duration(self):
         self.assertEqual(parse_duration('PT1H0.040S'), 3600.04)
         self.assertEqual(parse_duration('PT00H03M30SZ'), 210)
         self.assertEqual(parse_duration('P0Y0M0DT0H4M20.880S'), 260.88)
+        self.assertEqual(parse_duration('01:02:03:050'), 3723.05)
+        self.assertEqual(parse_duration('103:050'), 103.05)
 
     def test_fix_xml_ampersands(self):
         self.assertEqual(