Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#6831 closed defect (fixed)

Wrong aspect Ratio on some videos with internal player

Reported by: yianniv Owned by: markk
Priority: minor Milestone: unknown
Component: MythTV - Video Playback Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Some video files that don't have a 16/9 aspect ratio, are shown stretched vertically (long faces...:).

The mediainfo program in openSuSE reports that they have e.g.

720x432 (2.407 aspect ratio)

720x416 (2.450)

720x432 (2.35)

to name just three of them.

Xine plays them with the correct ratio fullscreen.

This happens on both a Debian and an openSuSE 11.1 frontends. I am including the output from mythfrontend --version on both systems as well as the mediainfo report on one of them. Anything else I should include?


openSuSE mythfrontend --version:

MythTV Version : 21203 MythTV Branch : trunk Network Protocol : 46 Library API : 0.22.20090804-1 QT Version : 4.5.2 Options compiled in:

linux release using_oss using_alsa using_arts using_jack using_backend using_directfb using_dvb using_frontend using_hdpvr using_iptv using_ivtv using_libfftw3 using_lirc using_mheg using_opengl_video using_opengl_vsync using_qtwebkit using_v4l using_x11 using_xrandr using_xv using_xvmc using_xvmc_vld using_xvmcw using_bindings_perl using_bindings_python using_opengl using_vdpau using_ffmpeg_threads using_live using_mheg


Debian mythfrontend --version: MythTV Version : 21203M MythTV Branch : trunk Network Protocol : 46 Library API : 0.22.20090804-1 QT Version : 4.4.3 Options compiled in:

linux release using_oss using_alsa using_arts using_jack using_backend using_directfb using_dvb using_firewire using_frontend using_hdpvr using_iptv using_ivtv using_libfftw3 using_lirc using_mheg using_opengl_video using_opengl_vsync using_qtwebkit using_v4l using_x11 using_xrandr using_xv using_xvmc using_xvmc_vld using_xvmcw using_bindings_perl using_bindings_python using_opengl using_vdpau using_ffmpeg_threads using_libavc_5_3 using_live using_mheg


mediainfo Complete name : Video1.mkv Format : Matroska File size : 2.26 GiB Duration : 2h 14mn Overall bit rate : 2 189 Kbps Writing application : HandBrake? svnexported

Video ID : 1 Format : AVC Format/Info? : Advanced Video Codec Format profile : High@L3.0 Format settings, CABAC : Yes Format settings, ReFrames? : 3 frames Muxing mode : Container profile=Unknown@3.0 Codec ID : V_MPEG4/ISO/AVC Duration : 2h 14mn Bit rate : 1 762 Kbps Nominal bit rate : 1 800 Kbps Width : 720 pixels Height : 432 pixels Display aspect ratio : 2.35 Frame rate : 25.000 fps Resolution : 24 bits Colorimetry : 4:2:0 Scan type : Progressive Bits/(Pixel*Frame) : 0.227 Stream size : 1.82 GiB (80%) Writing library : x264 core 65 r1028M 83baa7f Encoding settings : cabac=1 / ref=3 / deblock=1:0:0 / analyse=0x3:0x133 / me=umh / subme=9 / psy_rd=1.0:1.0 / mixed_ref=1 / me_range=16 / chroma_me=1 / trellis=1 / 8x8dct=1 / cqm=0 / deadzone=21,11 / chroma_qp_offset=-4 / threads=3 / nr=0 / decimate=1 / mbaff=0 / bframes=6 / b_pyramid=1 / b_adapt=1 / b_bias=0 / direct=3 / wpredb=1 / keyint=250 / keyint_min=25 / scenecut=40(pre) / rc=2pass / bitrate=1800 / ratetol=1.0 / qcomp=0.60 / qpmin=10 / qpmax=51 / qpstep=4 / cplxblur=20.0 / qblur=0.5 / ip_ratio=1.40 / pb_ratio=1.30 / aq=1:1.00

Audio ID : 2 Format : AC-3 Format/Info? : Audio Coding 3 Codec ID : A_AC3 Duration : 2h 14mn Bit rate mode : Constant Bit rate : 384 Kbps Channel(s) : 6 channels Channel positions : Front: L C R, Surround: L R, LFE Sampling rate : 48.0 KHz Stream size : 406 MiB (18%) Language : English

Attachments (1)

playback.log.gz (4.3 KB) - added by yianniv 10 years ago.
output of mythfrontend -v playback

Download all attachments as: .zip

Change History (9)

comment:1 Changed 10 years ago by markk

Owner: changed from Isaac Richards to markk
Status: newaccepted

Can you please attach the full output of of mythfrontend -v playback and I will probably need a short clip to test with as well (10 seconds or so should be fine). Thank.

Changed 10 years ago by yianniv

Attachment: playback.log.gz added

output of mythfrontend -v playback

comment:2 Changed 10 years ago by yianniv

Thanks for your reply, can you tell me how to include a video sample? A 17 sec sample is 5.1Mb.

comment:3 Changed 10 years ago by robertm

Hi Yianni,

You can upload it to one of many file share sites or a web site of your choosing. Rapidshare, etc. may be your quickest option.

comment:4 Changed 10 years ago by yianniv

Robert, thanks for your input. I thought there would be a more "direct" way.

Here is the link:

http://rapidshare.com/files/267803091/output-060.mkv.html

comment:5 Changed 10 years ago by markk

Yianni

Thanks for the log and clip. I've confirmed the issue here. For some reason, mythtv isn't picking up the aspect ratio and seems to be defaulting to assuming 'square' pixels. I suspect this is related to the matroska container format but will need to do some further investigation.

If anyone else has experience with mkv files, please pitch in!

Mark

comment:6 Changed 10 years ago by robertm

Hi Mark,

The issue appears to be in our defining aspect_ratio in avformatdecoder. ffplay does so correctly here, but we seem to reckon aspect with some code borrowed from a very old version. MKV contains both a pixel w/h and a display w/h. sample_aspect_ratio in the case of matroska appears to return pixel w/h. I am hoping this may help:

http://www.mail-archive.com/ffmpeg-soc@mplayerhq.hu/msg00146.html http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/2008-August/051987.html And also ffplay.c.

Robert

comment:7 Changed 10 years ago by Janne Grunau

Resolution: fixed
Status: acceptedclosed

(In [21308]) Take demuxer's sample aspect ratio into account. Fixes #6831

comment:8 Changed 10 years ago by Nigel

(In [21562]) Bug fix in aspect ratio change [21308]. Thanks to Mark. Closes #6925. Refs #6831. Now, if only I could work out why this (e.g. having an initial aspect ratio of 1.33333 instead of 1.77778 for 16:9 material) only impacts the Mac frontend? Maybe related to #6211

Note: See TracTickets for help on using tickets.