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 651564 - [vorbisenc/theoraenc] Proxy downstream caps restrictions (e.g. resolution, channels, etc)
[vorbisenc/theoraenc] Proxy downstream caps restrictions (e.g. resolution, ch...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.10.36
Assigned To: Luis de Bethencourt
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-05-31 16:41 UTC by Sebastian Dröge (slomo)
Modified: 2012-03-23 17:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
https://github.com/luisbg/gst-plugins-base/commit/2eb7d899ca56f3bdfd4de34ac5fcc003de2f939c (3.79 KB, patch)
2011-06-02 17:59 UTC, Luis de Bethencourt
none Details | Review
theora: get sink caps info from downstream element pad (3.79 KB, patch)
2011-06-02 18:02 UTC, Luis de Bethencourt
committed Details | Review
theora: set the width/height/par on the srcpad caps (2.21 KB, patch)
2011-06-02 18:02 UTC, Luis de Bethencourt
committed Details | Review
theora: use fixed src cap pads (897 bytes, patch)
2011-06-02 18:03 UTC, Luis de Bethencourt
committed Details | Review

Description Sebastian Dröge (slomo) 2011-05-31 16:41:01 UTC
See summary, currently both encoders don't do that. I guess the same applies for many other encoders, especially ones in other modules, too. x264enc and xvidenc are fixed though.

Reason why we need this is that it allows to use the constraints of encodebin and in general allows better negotiation.
Comment 1 Luis de Bethencourt 2011-05-31 16:46:13 UTC
essentially this means that a getcaps function on the sinkpad needs to be implemented, getting the downstream peer caps and doing something useful with them (i.e. converting them to the upstream format)

see x264enc/xvidenc for example
Comment 2 Sebastian Dröge (slomo) 2011-06-01 06:35:46 UTC
Also see bug #584599
Comment 4 Luis de Bethencourt 2011-06-02 18:00:31 UTC
theora: get sink caps info from downstream element pad 

meant that ^, instead of the url above
Comment 5 Luis de Bethencourt 2011-06-02 18:02:01 UTC
Created attachment 189102 [details] [review]
theora: get sink caps info from downstream element pad 

patch 001/003
Comment 6 Luis de Bethencourt 2011-06-02 18:02:56 UTC
Created attachment 189103 [details] [review]
theora: set the width/height/par on the srcpad caps 

patch 002/003
Comment 7 Luis de Bethencourt 2011-06-02 18:03:27 UTC
Created attachment 189105 [details] [review]
theora: use fixed src cap pads 

patch 003/003
Comment 8 Luis de Bethencourt 2011-06-02 18:04:23 UTC
thanks for all the help Sebastian
Comment 9 Vincent Penquerc'h 2012-02-03 12:19:07 UTC
All patches are marked as committed, is there any issue pending, or can this be closed ?
Comment 10 Akhil Laddha 2012-03-23 16:00:26 UTC
Luis, Sebastian, can this bug be closed ?
Comment 11 Luis de Bethencourt Guimera 2012-03-23 17:07:29 UTC
Yes Akhil, it can be closed.

I don't have the priviledges to do so.