[FFmpeg-devel] [PATCH 4/6] dv: fix weight table for 2x4x8 transform

[Christophe Gisquet]

Hi,

2014-10-30 14:49 GMT+01:00 Christophe Gisquet <christophe.gisquet at gmail.com>:
> Hi,
>
> 2014-10-26 19:20 GMT+01:00 Michael Niedermayer <michaelni at gmx.at>:
>> I took the liberty to apply the patch and fix the bug instead of
>> leaving it open until someone succeeds writing a fate test for it
>
> Hi, I wonder whether this should be reverted for now (and ticket #2970
> reopened at the same time).
>
> I have been checking how to produce interlaced content through the
> encoder, and the current table produces worse results (PSNR-wise, but
> it was big enough to not bother checking visually).
>
> I don't know if that's an encoder issue yet or if the decoder table is
> still incorrect.

So I've rederived the encoder table, and I'm getting the following
PSNR values, using tiny_psnr, on a progressive sequence from a chained
encoding and decoding:
Full progressive: stddev:    0.74 PSNR: 50.69 MAXDIFF:   19
Before:           stddev:    1.43 PSNR: 44.97 MAXDIFF:   17
This patch:       stddev:    0.76 PSNR: 50.44 MAXDIFF:   14

-- 
Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-dvenc-fix-2x4x8-interlaced-weight-table.patch
Type: text/x-patch
Size: 2251 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150131/5ccb0419/attachment.bin>