Better default for yt-dlp

[donob4n]

I know that you can configure desired size on "ytdl-format" but I use mpv on many different computers and containers. It's nice to run 'mpv https://www.youtube.com/watch?v=xxxxxxxxxx' just after install (like an integrated freature) but then I notice it's playing a 4k video on a FullHD monitor and it's a waste of resources.

For yt-dlp is a reasonable default because you are downloading a video locally which could then be played on many devices, but running directly from mpv makes more sense to get the best adjusted to your device.

I looked at "ytdl_hook.lua" but it seems that there is no easy way to get the resolution of the screen from it (maybe calling external programs which is not very nice...).

What do you think? 😅

[Argon-]

On the contrary, the display resolution is available as property in lua scripts: https://mpv.io/manual/master/#command-interface-display-height

Instead of modifying internal scripts, it may be easier to use conditional profiles and create e.g. two profiles for ytdl-format settings: one for >1080 and one for <= 1080

[donob4n]

I think that the resolution is not avaiable until start playing the video so not sure if yt-dlp could get that info before start downloading.

The "profile" solution means doing manual config? The idea is to have a reasonable default so don't do any intervention.

[donob4n]

@HedioKojima could you say what it's exactly the problem?