[FFmpeg-devel] ffmpeg -s breaks aspect ration (regression)

[Jean Delvare]

Michael,

On Tue, 16 Aug 2011 17:52:59 +0200, Jean Delvare wrote:
> On Mon, 1 Aug 2011 11:42:13 +0200, Michael Niedermayer wrote:
> > On Wed, Jul 20, 2011 at 04:44:56PM +0200, Jean Delvare wrote:
> > > My input video looks like:
> > > 
> > > Stream #0.0[0x1e0]: Video: mpeg2video (Main), yuv420p, 720x576 [PAR 64:45 DAR 16:9], 9500 kb/s, 25 fps, 25 tbr, 90k tbn, 50 tbc
> > > 
> > > I am passing -s 640x360 to ffmpeg to get square pixels, before the bug
> > > I would get:
> > > 
> > > Stream #0.0(und): Video: mpeg4, yuv420p, 640x360 [PAR 1:1 DAR 16:9], 640 kb/s, 25 fps, 25 tbr, 25 tbn, 25 tbc
> > > 
> > > and the video would look OK. Since the commit above, I get:
> > > 
> > > Stream #0.0(und): Video: mpeg4, yuv420p, 640x360 [PAR 1:1 DAR 16:9], 743 kb/s, PAR 91:64 DAR 91:36, 25 fps, 25 tbr, 25 tbn, 25 tbc
> > 
> > This bug seems to have been fixed as i cannot reproduce this anymore
> 
> Really? I still can, with the latest ffmpeg straight from git. See by
> yourself:

I can confirm that this is fixed now, I can no longer reproduce the bug
with the git master version as of today. Bisection points to this
commit as being the fix:

commit c8868f28e357e7e6ffe3254d0056b3e8033fe8e5
Author: Michael Niedermayer <michaelni at gmx.at>
Date:   Sat Aug 27 01:49:55 2011 +0200

    vf_scale: apply the same transform to the aspect during init that is applied per frame
    
    Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

So thanks Michael for solving my problem :) Can you please also
backport this fix to the 0.8 branch? Version 0.8.2 is affected by the
bug too.

Thanks again,
-- 
Jean Delvare