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 740072 - h263parse: ignores width/height on the sink pad and produces wrong size
h263parse: ignores width/height on the sink pad and produces wrong size
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other All
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-11-13 17:16 UTC by Josep Torra Valles
Modified: 2018-11-03 13:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Respect size in upstream caps (1.93 KB, patch)
2016-04-15 18:06 UTC, Josep Torra Valles
none Details | Review

Description Josep Torra Valles 2014-11-13 17:16:56 UTC
Run the following to reproduce the issue:

gst-launch-1.0 playbin uri=http://samples.mplayerhq.hu/V-codecs/h263/100374.mov

In the sink pad video size is 240x180 and in the src pad is declared 352x288.

The correct size is 240x180 as it's specified in the container.

/GstPlayBin:playbin0/GstURIDecodeBin:uridecodebin0/GstDecodeBin:decodebin0/GstH263Parse:h263parse0.GstPad:sink: caps = "video/x-h263\,\ variant\=\(string\)itu\,\ width\=\(int\)240\,\ height\=\(int\)180\,\ framerate\=\(fraction\)12000/1001\,\ pixel-aspect-ratio\=\(fraction\)1/1"
Redistribute latency...
/GstPlayBin:playbin0/GstURIDecodeBin:uridecodebin0/GstDecodeBin:decodebin0/avdec_h263:avdec_h263-0.GstPad:sink: caps = "video/x-h263\,\ variant\=\(string\)itu\,\ width\=\(int\)352\,\ height\=\(int\)288\,\ framerate\=\(fraction\)12000/1001\,\ pixel-aspect-ratio\=\(fraction\)1/1\,\ parsed\=\(boolean\)true\,\ annex-d\=\(boolean\)true\,\ annex-e\=\(boolean\)false\,\ annex-f\=\(boolean\)false\,\ annex-g\=\(boolean\)false\,\ annex-i\=\(boolean\)false\,\ annex-j\=\(boolean\)false\,\ annex-k\=\(boolean\)false\,\ annex-m\=\(boolean\)false\,\ annex-n\=\(boolean\)false\,\ annex-q\=\(boolean\)false\,\ annex-r\=\(boolean\)false\,\ annex-s\=\(boolean\)false\,\ annex-t\=\(boolean\)false\,\ annex-u\=\(boolean\)false\,\ annex-v\=\(boolean\)false\,\ profile\=\(string\)1"
/GstPlayBin:playbin0/GstURIDecodeBin:uridecodebin0/GstDecodeBin:decodebin0/GstH263Parse:h263parse0.GstPad:src: caps = "video/x-h263\,\ variant\=\(string\)itu\,\ width\=\(int\)352\,\ height\=\(int\)288\,\ framerate\=\(fraction\)12000/1001\,\ pixel-aspect-ratio\=\(fraction\)1/1\,\ parsed\=\(boolean\)true\,\ annex-d\=\(boolean\)true\,\ annex-e\=\(boolean\)false\,\ annex-f\=\(boolean\)false\,\ annex-g\=\(boolean\)false\,\ annex-i\=\(boolean\)false\,\ annex-j\=\(boolean\)false\,\ annex-k\=\(boolean\)false\,\ annex-m\=\(boolean\)false\,\ annex-n\=\(boolean\)false\,\ annex-q\=\(boolean\)false\,\ annex-r\=\(boolean\)false\,\ annex-s\=\(boolean\)false\,\ annex-t\=\(boolean\)false\,\ annex-u\=\(boolean\)false\,\ annex-v\=\(boolean\)false\,\ profile\=\(string\)1"
Comment 1 Josep Torra Valles 2016-04-15 18:06:15 UTC
Created attachment 326123 [details] [review]
Respect size in upstream caps
Comment 2 Sebastian Dröge (slomo) 2016-04-16 09:01:02 UTC
There's a similar bug somewhere about the same with h264. IMHO we should use the crop meta for this. The in-codec dimensions are like they are on the caps now, just that it should be cropped.
Comment 3 Sebastian Dröge (slomo) 2016-04-16 09:08:17 UTC
See bug #685812, bug #693971 and bug #694068
Comment 4 GStreamer system administrator 2018-11-03 13:28:17 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/188.