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

[Michael Niedermayer]

On Tue, Sep 06, 2011 at 11:17:45PM +0200, Jean Delvare wrote:
> 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.

ill try to backport that commit to 0.8, but i dont know if thats enough

thanks

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

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- 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/20110906/22f59399/attachment.asc>