Re: [FFmpeg-devel] [PATCH] avcodec/svq1: fix interframe mean VLC symbols

2022-10-18 Thread Michael Niedermayer
On Tue, Oct 18, 2022 at 12:44:20PM +1100, Peter Ross wrote: > On Mon, Oct 17, 2022 at 10:38:53PM +0200, Michael Niedermayer wrote: > > On Mon, Oct 17, 2022 at 08:33:28PM +1100, Peter Ross wrote: > > > On Mon, Oct 17, 2022 at 05:04:29AM +0200, Andreas Rheinhardt wrote: > > > > Peter Ross: > > > > >

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: fix interframe mean VLC symbols

2022-10-17 Thread Peter Ross
On Mon, Oct 17, 2022 at 10:38:53PM +0200, Michael Niedermayer wrote: > On Mon, Oct 17, 2022 at 08:33:28PM +1100, Peter Ross wrote: > > On Mon, Oct 17, 2022 at 05:04:29AM +0200, Andreas Rheinhardt wrote: > > > Peter Ross: > > > > Fixes ticket #128. > > > > > > > > The SVQ1 interframe mean VLC symbo

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: fix interframe mean VLC symbols

2022-10-17 Thread Michael Niedermayer
On Mon, Oct 17, 2022 at 08:33:28PM +1100, Peter Ross wrote: > On Mon, Oct 17, 2022 at 05:04:29AM +0200, Andreas Rheinhardt wrote: > > Peter Ross: > > > Fixes ticket #128. > > > > > > The SVQ1 interframe mean VLC symbols -128 and 128 are incorrectly swapped > > > in our SVQ1 implementation, resulti

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: fix interframe mean VLC symbols

2022-10-17 Thread Peter Ross
On Mon, Oct 17, 2022 at 05:04:29AM +0200, Andreas Rheinhardt wrote: > Peter Ross: > > Fixes ticket #128. > > > > The SVQ1 interframe mean VLC symbols -128 and 128 are incorrectly swapped > > in our SVQ1 implementation, resulting in visible artifacts for some videos. > > This patch unswaps the orde

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: fix interframe mean VLC symbols

2022-10-16 Thread Andreas Rheinhardt
Peter Ross: > Fixes ticket #128. > > The SVQ1 interframe mean VLC symbols -128 and 128 are incorrectly swapped > in our SVQ1 implementation, resulting in visible artifacts for some videos. > This patch unswaps the order of these two symbols. > > The most noticable example of the artiacts caused b

[FFmpeg-devel] [PATCH] avcodec/svq1: fix interframe mean VLC symbols

2022-10-16 Thread Peter Ross
Fixes ticket #128. The SVQ1 interframe mean VLC symbols -128 and 128 are incorrectly swapped in our SVQ1 implementation, resulting in visible artifacts for some videos. This patch unswaps the order of these two symbols. The most noticable example of the artiacts caused by this error can be observ