SVT-VP9 icon indicating copy to clipboard operation
SVT-VP9 copied to clipboard

Why is the ffmpeg patch not already in ffmpeg master?

Open thesn10 opened this issue 3 years ago • 6 comments

I can install SVT-AV1 without having to manually patch ffmpeg. So why isnt this the case with SVT-VP9 already?

thesn10 avatar Mar 14 '21 15:03 thesn10

It requires a lot of effort and cleanup to upstream the patch and we do not have the resources and time to do so at this moment.

1480c1 avatar Mar 14 '21 17:03 1480c1

Looks like somebody asked about this 2 years ago already: https://trac.ffmpeg.org/ticket/7914 .

rathann avatar Apr 13 '21 11:04 rathann

I'm in the process of updating the patch for ffmpeg n6.0 now. I'll post a link here if I get it working. I had to make some adjustments that I'm not 100% certain about.

joeyparrish avatar Sep 11 '23 23:09 joeyparrish

I'm giving up for now. I got it to build, but I get this error:

[0911/173548:ERROR:demuxer.cc(202)] Failed to detect the container type from the buffer: 000000206674797069736F6D0000020069736F6D69736F3669736F326D703431

I don't get a similar error from SVT-AV1.

Here's the patch I'm using.
n6.0-0001-Add-ability-for-ffmpeg-to-run-svt-vp9.patch.txt

My guess is that I messed up the changes to the mkv_write_block() calls. That function changed so much that I was really not sure I was mapping the changes in the patch correctly.

joeyparrish avatar Sep 12 '23 00:09 joeyparrish

This simplified version has the same issue:

n6.0-0001-Simplified-libsvtvp9.patch.txt

I hope these are a useful starting point for someone who knows more than me about SVT-VP9 and FFmpeg both.

joeyparrish avatar Sep 12 '23 00:09 joeyparrish

Oh, crap, I was starting from the patch at SVT-VP9 0.30, which has been revised since then. Disregard the above.

joeyparrish avatar Sep 12 '23 01:09 joeyparrish