youtube-dl (original) (raw)

In some issue report someone commented saying that for videos in low resolution it's more convenient to just get the normal formats, since they have the same or better quality. Checking these cases would probably complicate the code too much and I don't think it's worth the effort.

Potential merge issues

We should make sure that the default formats work (otherwise users would get confused), I propose two alternatives:

If bestaudio and bestvideo are not compatible, we merge into a mkv file
Use bestvideo[mp4]+bestaudio[m4a]/bestvideo[webm]+bestaudio[webm]/mp4 instead, with this option we may not get the best quality (I think in some cases there's either a webm or mp4 with better resolution, but I'm not really sure).

With this change the YoutubeIE test fails, we have to change the format field in test/parameters.json.

Apart from the previous comments, I like the change and agree that bestvideo+bestaudio/best should be the default

[YoutubeDL] Make bestvideo+bestaudio/best default format when merger is available by dstftw · Pull Request #5456 · ytdl-org/youtube-dl (original) (raw)

[YoutubeDL] Make `bestvideo+bestaudio/best` default format when merger is available by dstftw · Pull Request #5456 · ytdl-org/youtube-dl (original) (raw)