Houmeres/SageTube
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 4 Wiki Activity

Improve playlist format selection and resolution

Browse Source 
When downloading playlists, avoid reusing a specific format_id from the first item (which can fail for subsequent videos). For audio-only playlists, use yt-dlp's bestaudio fallback; for video playlists, respect a provided resolution by extracting the height and applying -S res:<height>, otherwise fall back to bestvideo+bestaudio/best. Invalid resolution strings are caught and logged, and revert to the dynamic best-quality fallback.
This commit is contained in:
oop7
2026-04-28 12:55:46 +03:00
parent bb461898e4
commit fe0f8241ff
1 changed files with 23 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ytsage/core/ytsage_downloader.py
+23 -1
View File
@@ -242,7 +242,29 @@ class DownloadThread(QThread):
logger.debug(f"Using {self.concurrent_fragments} concurrent connections")
# Format selection strategy - use format ID if provided or fallback to resolution
if self.format_id:
if self.is_playlist:
# For playlists, specific format_id from the first video often fails for subsequent videos.
# Instead, we rely on dynamic fallback/resolution limits.
if self.is_audio_only:
# For audio-only playlist, let yt-dlp pick best audio.
cmd.extend(["-f", "bestaudio/best"])
logger.debug(f"Playlist mode: using dynamic best audio fallback instead of format_id")
else:
# If a specific resolution is given, limit to it. Otherwise, select the overall best.
# The resolution might be e.g. "1920x1080" or "1080". We want the height.
try:
if self.resolution and self.resolution != "default":
res_str = str(self.resolution)
h = min(map(int, res_str.split('x'))) if 'x' in res_str else int(res_str)
cmd.extend(["-S", f"res:{h}"])
logger.debug(f"Playlist mode: using resolution limiter -S res:{h}")
else:
cmd.extend(["-f", "bestvideo+bestaudio/best"])
logger.debug("Playlist mode: using dynamic best quality overall")
except ValueError:
cmd.extend(["-f", "bestvideo+bestaudio/best"])
logger.debug("Playlist mode: invalid resolution string, using dynamic best quality overall")
elif self.format_id:
clean_format_id: str = self.format_id.split("-drc")[0] if "-drc" in self.format_id else self.format_id
# If the selected format is audio-only, pass it directly.