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 596326 - [qtdemux] wrong display_width/display_height
[qtdemux] wrong display_width/display_height
Status: RESOLVED DUPLICATE of bug 679522
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-09-25 16:07 UTC by Philippe Normand
Modified: 2013-08-21 18:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
sample media file (199.08 KB, video/quicktime)
2009-09-25 16:07 UTC, Philippe Normand
Details
qtdemux mem dump (101.23 KB, text/plain)
2009-09-25 16:08 UTC, Philippe Normand
Details

Description Philippe Normand 2009-09-25 16:07:44 UTC
Created attachment 144014 [details]
sample media file

Attaching a file being played by Quicktime at 640x480 and recognized by
GStreamer as 320x240
Comment 1 Philippe Normand 2009-09-25 16:08:21 UTC
Created attachment 144015 [details]
qtdemux mem dump
Comment 2 Mark Nauwelaerts 2009-09-25 17:30:01 UTC
I have no idea as to why Quicktime feels it should play this 640x480.  GStreamer, mplayer, vlc all play this at 320x240 and qtinfo only reports the following:
Type: Quicktime
  0 audio tracks.
  1 video tracks.
    320x240, depth 24
    rate 3.015075 [600:199] constant
    length 30 frames
    compressor png .
    Native colormodel:  24 bpp RGB
    Interlace mode:     None (Progressive)
    supported.
  0 text tracks.
Comment 3 Michael Smith 2009-09-25 18:07:21 UTC
I suspect this is from the display matrix in the mvhd atom. In this file, it is (I think - manually parsing it from a hex dump, I haven't written code for this):

[ 2   0   0 ]
[ 0   2   0 ]
[ 0   0   1 ]

Also, even if we did parse this, we don't actually have any mechanism in gstreamer to expose an arbitrary transformation matrix to downstream.
Comment 4 Philippe Normand 2009-09-26 17:55:43 UTC
Taking in account the scale matrix, couldn't the size of the video be updated? This is a loss of information though, the original size will be lost :/
Comment 5 Michael Smith 2009-09-26 23:33:20 UTC
Phil: no, you can't do that. The only place you could change the size is when you actually scale the video - in videoscale for example - and there's no way to get the information (currently) to videoscale - you can only express the _actual_ size of the video, not how you want it displayed.
Comment 6 Sebastian Dröge (slomo) 2009-09-27 09:28:41 UTC
(In reply to comment #5)
> Phil: no, you can't do that. The only place you could change the size is when
> you actually scale the video - in videoscale for example - and there's no way
> to get the information (currently) to videoscale - you can only express the
> _actual_ size of the video, not how you want it displayed.

Well, actually you can by using the p-a-r field. But because the core is dividing the numerator and denominator by the gcd this doesn't work in this case. If p-a-r of 2/2 was allowed and possible, videoscale could do the right thing.

(but that's no pixel aspect *ratio* anymore... more a pixel size)
Comment 7 Jan Schmidt 2009-09-27 11:00:51 UTC
pixel-aspect-ratio is the wrong place to attempt to put the information - even though they're related.

I imagine that it's possible to have a QT file that has both pixel-aspect-ratio *and* a desired presentation transformation matrix.
Comment 8 Philippe Normand 2009-09-28 08:25:34 UTC
Are there other formats and/or codecs that provide this kind of information (transformation matrix)? If so maybe it'd be worth some addition in the core?
Comment 9 Tim-Philipp Müller 2012-10-26 19:31:05 UTC
Could probably do something clever with metas now, though I'm not entirely sure what the point is and if we shouldn't just WONTFIX it, or perhaps just post an element message for the application so it can take care of that if it wants to?
Comment 10 Philippe Normand 2012-10-29 15:57:19 UTC
Dupe of Bug 679522 ?
Comment 11 Sebastian Dröge (slomo) 2013-08-21 18:18:39 UTC

*** This bug has been marked as a duplicate of bug 679522 ***