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

Tim Nicholson nichot20 at yahoo.com
Tue Apr 3 18:07:10 CEST 2012

On 30/03/12 15:47, Michael Niedermayer wrote:
> 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

I have checked all the fate results. In every case the handling of
progressive material within an interlaced stream has led to vertical
softening of the final material leading to differences in the checksums.

This resubmitted version is the same as the original, but includes the
updated checksums.

For interlaced material I have tested using the same settings as fate.
The results show the integrity of the interlacing is maintained leading
to an improvement in the final results.

To make best use of this patch more accurate interlace detection is
required, perhaps based upon picture content and not flag settings.

Whilst this is beyond the scope of this patch it is already a work in
progress by others.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Make-default-setting-of-parameter-interl-1-auto.patch
Type: text/x-patch
Size: 7041 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120403/5cdb76ef/attachment.bin>

More information about the ffmpeg-devel mailing list