Thumbnailing failed on some youtube DASH videos about mpv_thumbnail_script HOT 8 CLOSED

Fixed in 40cadfc.

from mpv_thumbnail_script.

oM36vTQOkP0 creates an edl:// (a... virtual file which can join several sources together) way too long to pass in a command line argument. I get 145451 bytes of it. 7rG7wPCcxmw is reasonably sized.

from mpv_thumbnail_script.

Hmh. I thought I could use a playlist to bypass the limit but mpv can only handle lines up to 8K characters long.
Sadly I'll have to allow mpv pass the original url through ytdl again. It'll really slow down the process, but eh.

from mpv_thumbnail_script.

Thx for the fast response again!

from mpv_thumbnail_script.

Hmm... Have you looked at the -a parameter of youtube-dl?

       -a, --batch-file FILE
              File containing URLs to download ('-' for stdin)

You can pass the url from file or stdin, and it doesn't have length limit.

from mpv_thumbnail_script.

The edl:// is not for youtube-dl, it's for mpv. I don't call ytdl itself ever, and even if stdin was the solution, it's not because it's a pain to try to pipe stuff in a cross-platform manner (because mpv doesn't open a shell).

The real solution is to have a script setup a pre-hook and let the script read the playlist with no character limits, but that's not a clean solution yet (workers should not pre-hook because there can be multiple, and the osc-client pre-hooking feels wrong and having a third script just to pre-hook the playlist is stupid).

from mpv_thumbnail_script.

or you can make a patch for mpv, or ask for a feature/changing in code.

from mpv_thumbnail_script.

150 kilobytes of a media path is ridiculous. It's not fair to expect mpv to parse playlist entries that long, and I have features I'd already want in that they're not really interested in - a patch to read up to and over 150 kilobyte lines would be scoffed at - there's just no reason to expect entries that long, especially so since this can be fixed with the hook script.
Plus then the functionality of the script would depend on the mpv version. Not all users have a cronjob for daily mpv-build sessions.

from mpv_thumbnail_script.