[FFmpeg-devel] Initial tickering at unifying ProRes encoder
Clément Bœsch
u at pkh.me
Wed Jan 10 15:22:52 EET 2024
On Sun, Dec 24, 2023 at 11:56:56AM +0100, Stefano Sabatini wrote:
> On date Monday 2023-12-11 02:35:01 +0100, Clément Bœsch wrote:
> > Hello there,
> >
> > I'm currently investigating an issue¹ affecting Kostya ProRes encoder
> > which is not reproducible with Anatoliy encoder (but it's too early to
> > say if the bug is not present there as well given the differences in
> > quantization, and it's also unclear if the issue is even actually
> > there in the first place and not in M2/M3 firmwares).
> >
> > Since these encoders share more than similiarities, I've been looking
> > into their differences, and unifying their common codepaths was helpful
> > to my investigation.
> >
> > Attached is an initial patchset working out their differences by either
> > sharing data or making code identical where relevant.
> >
> > I will continue that work as the investigation progresses. The
> > quantization work might require more involved changes though.
> >
> > Regards,
> >
> > ¹ https://trac.ffmpeg.org/ticket/10255
>
> Hi Clement.
>
> Most patches look good to me. You can skip style comments in patches
> 3, 4, 15 if they complicate the rebase.
Patchset applied; I skipped the 2 out of the 3 cosmetics
Thank you for the review
--
Clément B.
More information about the ffmpeg-devel
mailing list