[mythtv] mythtv commit: r8651 by danielk (Auto Aspect Ratio transcode)
Endaf Jones
jonese at zener.com
Thu Jan 19 23:53:13 UTC 2006
> On 19 Jan 2006, at 22:42, Endaf Jones wrote:
>
> >> Modified:
> >>
> >> trunk/mythtv/libs/libmythtv/videobuffers.cpp
> >>
> >> Log:
> >>
> >> Fixes #1053.
> >>
> >> In this case an MPEG-4 recording is using a frame height which is not
> >> a multiple of 16. ffmpeg isn't doing any validation on the dimensions
> >> so it ends up writing outside the buffer. This just pads allocated
> >> buffers up to multiples of 16, so that this type of error doesn't
> >> cause
> >> a segfault with any of the codecs we support. MythTV does prevent you
> >> from setting an invalid dimension when you record to a MPEG-4 NUV, so
> >> AFAIK this is more of a legacy issue.
> >
> > Question..
> >
> > What about transcoding MPEG2 into MPEG4 after a recording and you have
> > "Resize Video while transcoding" AND the newly developed "Preserve
> > Aspect Ratio of Video when resizing" option? The height is auto
> > calculated in that case.
> >
> > Is this situation the same as the above commit?
> >
> > I say this because I have a PBS HDTV program 1920x1080i that I
> > transcode into a 544 wide MPEG4 so I can view it on my SDTV. The
> > height works out to be 302 or so pixels (to keep the aspect correct).
> > When I play that newly encoded file, I have a strange color cast on
> > the top 1/6th of the picture.
> >
> > Same problem or different?
> >
> > --
> > # Endaf
On 1/19/06, Dave Just <justifiably at cwazy.co.uk> wrote:
> Yes! This is just the situation I had. I have my low quality
> transcode settings to rescale from usual capture resolution of
> 720x576 down to 480x384. Both multiples of 16. But then I also
> added the "Preserve aspect option" when it appeared. The result was
> that some of my captures which are at 4:3 instead of 16:9 got scaled
> to a larger height, 392, not divisible by 16... and yes, the
> transcoded files have fuzzy green borders in the top 1/6th or so.
>
> I'm disabling the "preserve aspect ratio" for now again. Maybe that
> option needs tuning to enforce the standards. (BTW, wouldn't it be
> better to scale the width according to the desired height anyway? I
> think it's more useful to specify vertical resolution to match output
> device)
>
Glad to see I wasn't the only one with a strange color overlay at the top.
I also wondered if there is value in fixing the aspect ratio to the
height instead of the width (or have the option between the two).
--
# Endaf
More information about the mythtv-dev
mailing list