search

gfxdisp / ColorVideoVDP

Colour video VDP
MIT License
29 stars 6 forks source link

add support for reading Y4M input files #14

Closed cosmin closed 2 months ago

cosmin commented 4 months ago

pass -count_frames to ffprobe for y4m input since it does not contain the number of frames in header

Fixes #6

cosmin commented 4 months ago

cc @vibhoothi

vibhoothi commented 4 months ago

I totally missed -count_frames , it went top of my head^^,

This is smart way.

On a bit unrelated note, do you reckon when we do rgb48le conversions (vf_scale yuv -> rgb),ffmpeg would not touch the codevalues and we good to leave to ffmpeg for Y4Ms?

vibhoothi commented 2 months ago

I did test for prores streams, and it was substantially slower than defaults. So I would say this makes more sense

Vibhoothi, Sent from Mobile! mindfreeze.videolan.me

On Thu 5 Sep 2024 at 04:42, Rafal Mantiuk @.***> wrote:

Merged #14 https://github.com/gfxdisp/ColorVideoVDP/pull/14 into main.

— Reply to this email directly, view it on GitHub https://github.com/gfxdisp/ColorVideoVDP/pull/14#event-14139803876, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACSVASKWXSAKMNVT7SMZK4TZU7ALFAVCNFSM6AAAAABKVHUZLSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGEZTSOBQGM4DONQ . You are receiving this because you were mentioned.Message ID: @.***>