Merge pull request #105 from lrnselfreliance/bugfix/98-dont-download-live-videos

Using yt-dlp's match-filter to skip live videos.
This commit is contained in:
lrnselfreliance 2022-02-20 10:48:03 -07:00 • committed by GitHub
commit e4c3f7b9c8
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 5 additions and 5 deletions

View file

@ -163,6 +163,7 @@ class VideoDownloader(Downloader, ABC):
str(YT_DLP_BIN),
'-cw', # Continue downloads, do not clobber existing files.
'-f', PREFERRED_VIDEO_FORMAT,
'--match-filter', '!is_live', # Do not attempt to download Live videos.
'--write-subs',
'--write-auto-subs',
'--write-thumbnail',
@ -217,9 +218,8 @@ class VideoDownloader(Downloader, ABC):
raise ValueError(f'Output directory does not exist! {out_dir=}')
# YoutubeDL expects specific options, add onto the default options
config = get_downloader_config().dict()
options = config.copy()
options['outtmpl'] = f'{out_dir}/{config["file_name_format"]}'
options = get_downloader_config().dict()
options['outtmpl'] = f'{out_dir}/{options["file_name_format"]}'
options['merge_output_format'] = PREFERRED_VIDEO_EXTENSION
# We don't need "format" when getting the filename.
del options['format']

View file

@ -590,7 +590,7 @@ class DownloadManager:
return data
@optional_session
def get_pending_downloads(self, session: Session):
def get_pending_downloads(self, session: Session) -> List[Download]:
return session.query(Download).filter_by(status='pending').all()
@optional_session

View file

@ -1,4 +1,4 @@
__version__ = '0.3.1-beta'
__version__ = '0.3.2-beta'
import subprocess