[FFmpeg-devel] [PATCH] Change default behaviour of scale filter from 'progressive' to 'auto'

[Michael Niedermayer]

On Fri, Mar 30, 2012 at 10:51:28AM +0100, Tim Nicholson wrote:
> When the option of interlaced aware scaling was first introduced interlace detection
> was poor. This has been improved, and this proposed patch changes the default behaviour
> of the filter to depend upon the auto detected value, bringing it into line with
> many other options whose default value is -1 (auto).
> 
> From 4d1ad96e209b18bdf86530d153a6a5acbc5aab8a Mon Sep 17 00:00:00 2001
> From: Tim Nicholson <Tim.Nicholson at bbc.co.uk>
> Date: Fri, 30 Mar 2012 10:43:46 +0100
> Subject: [PATCH] Make default setting of parameter interl=-1 (auto)
> 
> Previously default was 0 (progressive), but interlace detection has been
> improved in 648e55ff, and this makes the behaviour consistent with other
> options that use 1|0|-1 with -1 the default.

This patch changes several fate tests checksums

please check if these changes are ok and if so update the checksums
with the patch


--- ./tests/ref/vsynth1/dnxhd_1080i 2012-03-26 11:38:07.476140754 +0200
+++ tests/data/fate/vsynth1-dnxhd_1080i 2012-03-30 16:35:39.321694082 +0200
@@ -1,4 +1,4 @@
 027c985483caab9397592bf27477dce1 *./tests/data/vsynth1/dnxhd-1080i.mov
 3031911 ./tests/data/vsynth1/dnxhd-1080i.mov
-0c651e840f860592f0d5b66030d9fa32 *./tests/data/dnxhd_1080i.vsynth1.out.yuv
-stddev:    6.29 PSNR: 32.15 MAXDIFF:   64 bytes:   760320/  7603200
+3c3226518a0f56468bf56a6682e31fae *./tests/data/dnxhd_1080i.vsynth1.out.yuv
+stddev:   14.22 PSNR: 25.07 MAXDIFF:  119 bytes:   760320/  7603200
Test vsynth1-dnxhd_1080i failed. Look at tests/data/fate/vsynth1-dnxhd_1080i.err for details.
make: *** [fate-vsynth1-dnxhd_1080i] Error 1
make: *** Waiting for unfinished jobs....
--- ./tests/ref/vsynth1/dv  2012-03-26 11:38:07.476140754 +0200
+++ tests/data/fate/vsynth1-dv  2012-03-30 16:35:40.329694054 +0200
@@ -1,4 +1,4 @@
 4d572f758b55a1756adf9f54132f3b9e *./tests/data/vsynth1/dv.dv
 7200000 ./tests/data/vsynth1/dv.dv
-02ac7cdeab91d4d5621e7ce96dddc498 *./tests/data/dv.vsynth1.out.yuv
-stddev:    6.90 PSNR: 31.34 MAXDIFF:   76 bytes:  7603200/  7603200
\ No newline at end of file
+31b3c8f0fa8ae336c561ccdee5738376 *./tests/data/dv.vsynth1.out.yuv
+stddev:   12.83 PSNR: 25.96 MAXDIFF:  125 bytes:  7603200/  7603200
Test vsynth1-dv failed. Look at tests/data/fate/vsynth1-dv.err for details.
make: *** [fate-vsynth1-dv] Error 1

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is what and why we do it that matters, not just one of them.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120330/84a98541/attachment.asc>