Re: [FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-12 Thread Christophe Gisquet
Hi, 2015-10-12 0:09 GMT+02:00 Michael Niedermayer : > this seems to cause artifacts on the red/cyan edge in > ./ffmpeg -f lavfi -i testsrc=s=1920x1080 -vframes 1 -pix_fmt yuv422p -vcodec > dnxhd -flags +ildct -vb 185M test.mov ok, ff_convert_matrix permutes the table, so the 8 bit case should us

Re: [FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 11:07:25PM +0200, Christophe Gisquet wrote: > Hi, > > 2015-10-11 18:20 GMT+02:00 Michael Niedermayer : > > On Sun, Oct 11, 2015 at 04:06:05PM +0200, Christophe Gisquet wrote: > >> The functions related to bitstream reading must use the natural zigzag > >> order, and not the

Re: [FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-11 Thread Christophe Gisquet
Hi, 2015-10-11 18:20 GMT+02:00 Michael Niedermayer : > On Sun, Oct 11, 2015 at 04:06:05PM +0200, Christophe Gisquet wrote: >> The functions related to bitstream reading must use the natural zigzag >> order, and not the one permuted for use in the iDCT. This currently >> results in bitstreams with

Re: [FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-11 Thread Michael Niedermayer
On Sun, Oct 11, 2015 at 04:06:05PM +0200, Christophe Gisquet wrote: > The functions related to bitstream reading must use the natural zigzag > order, and not the one permuted for use in the iDCT. This currently > results in bitstreams with 2 issues: > - AC coefficients are encoded in an unexpected

[FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-11 Thread Christophe Gisquet
The functions related to bitstream reading must use the natural zigzag order, and not the one permuted for use in the iDCT. This currently results in bitstreams with 2 issues: - AC coefficients are encoded in an unexpected order; - Incorrect weights are applied in the (de)quantization. This curren

[FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-09 Thread Christophe Gisquet
The functions related to bitstream reading must use the natural zigzag order, and not the one permuted for use in the iDCT. This currently results in bitstreams with 2 issues: - AC coefficients are encoded in an unexpected order; - Incorrect weights are applied in the (de)quantization. This curren

[FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-07 Thread Christophe Gisquet
The functions related to bitstream reading must use the natural zigzag order, and not the one permuted for use in the iDCT. This currently results in bitstreams with 2 issues: - AC coefficients are encoded in an unexpected order; - Incorrect weights are applied in the (de)quantization. This curren