After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 428686 - Aspect Ratio Distortion
Aspect Ratio Distortion
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
0.10.x
Other All
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-04-11 16:22 UTC by Cédric Marcone
Modified: 2008-10-13 13:45 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18


Attachments
normal screenshot (15.72 KB, image/jpeg)
2007-04-11 16:43 UTC, Cédric Marcone
Details
distorted screenshot (17.71 KB, image/jpeg)
2007-04-11 16:44 UTC, Cédric Marcone
Details

Description Cédric Marcone 2007-04-11 16:22:25 UTC
Please describe the problem:
I configured Xorg with two screens using mergedFB (i810 under Ubuntu Feisty). The layout is screen on pipeA (1280x800) above screen on pipeB (1280x1024). The result is a vertical virtual screen of 1280x1824.

Using this setup, the aspect ratio in totem-gstreamer is distorted (the output is higher than expected).


Steps to reproduce:
Use a vertical screen setup.

Actual results:
The image is higher than it should be.

Expected results:


Does this happen every time?
Yes

Other information:
I ran totem with GST_DEBUG=5 and isolated the following traces :

0:00:02.783701000 27531 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:437:get_media_size: display PAR is 5/3
0:00:02.783723000 27531 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:480:get_media_size: movie PAR is 16/15
0:00:02.783743000 27531 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:491:get_media_size: calculated scaling ratio 352/375 for video 352x240
0:00:02.783800000 27531 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:505:get_media_size: keeping video width
0:00:02.783818000 27531 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:516:get_media_size: scaling to 352x375

Then, I reverted to a mono screen setup (1280x800) and re-ran totem with the same movie. The aspect ratio is back to normal and the corresponding traces are :

0:00:02.805423000 27918 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:437:get_media_size: display PAR is 54/59
0:00:02.805445000 27918 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:480:get_media_size: movie PAR is 16/15
0:00:02.805466000 27918 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:491:get_media_size: calculated scaling ratio 10384/6075 for video 352x240
0:00:02.805495000 27918 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:510:get_media_size: approximating while keeping video height
0:00:02.805514000 27918 0x80940a0 DEBUG                totem bacon-video-widget-gst-0.10.c:516:get_media_size: scaling to 410x240
Comment 1 Cédric Marcone 2007-04-11 16:43:12 UTC
Created attachment 86182 [details]
normal screenshot

screenshot of the totem window in the mono screen setup
Comment 2 Cédric Marcone 2007-04-11 16:44:39 UTC
Created attachment 86183 [details]
distorted screenshot


screenshot of the totem window in the dual screen setup.
Comment 3 Philip Withnall 2007-06-10 13:01:30 UTC
GStreamer bug in gst_video_calculate_display_ratio.
Comment 4 Jan Schmidt 2007-06-13 09:13:47 UTC
What makes you think it's in gst_video_calculate_display_ratio? I'm more inclined to believe that the information X provides about the physical size of the display is wrong, causing xvimagesink to calculate a bogus pixel-aspect-ratio for the display.
Comment 5 Philip Withnall 2007-06-13 16:35:45 UTC
You're right. The display PAR does look a bit off. I would've thought this would affect more than just Totem, though...
Comment 6 Wim Taymans 2007-10-29 12:39:49 UTC
not many players actually do correct aspect ratio.