[gstreamer-bugs] [Bug 555699] [PATCH] theoradec: prefer container's pixel aspect ratio over the stream's

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=555699

  GStreamer | gst-plugins-base | Ver: HEAD CVS




------- Comment #3 from Robin Stocker  2008-10-10 10:08 UTC -------
Thanks for the quick and detailed response Michael!

Yes, that sounds like a good idea. Where the user didn't take care to set the
correct DAR for the container it would most likely be square or the same as the
stream itself (mkvmerge does it like that) anyway.

So, I looked at some files and found that my problems are mostly with MPEG
videos (from DVDs in 720x576). The stream itself has an incorrect 16:15 PAR
(4:3 DAR) but they are muxed in an MKV with the correct 64:45 PAR (16:9 DAR).
So if neither PAR is square, what should we do?

One solution is to take the PAR of the container. Another is to take the PAR
that is less square (64:45). Both solutions work in this case, but I like the
first better, because it's less arbitrary. I know there are many MPEG streams
with an incorrect PAR, but I don't think that there are many files where the
container specifies a different non-square PAR that is wrong. An argument for
the second solution is that there are more videos on YouTube where the heads
are too high than where the heads are too wide ;). What do you think?


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=555699.