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 776948 - Add vpx (de)muxing capabilities to mpegtsmux and mpegtsdemux
Add vpx (de)muxing capabilities to mpegtsmux and mpegtsdemux
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Windows
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-01-06 14:47 UTC by Yann Jouanin
Modified: 2018-11-03 14:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to add vpx support to both mpegtsmux and mpegtsdemux (13.46 KB, patch)
2017-01-06 14:47 UTC, Yann Jouanin
needs-work Details | Review

Description Yann Jouanin 2017-01-06 14:47:26 UTC
Created attachment 343024 [details] [review]
Patch to add vpx support to both mpegtsmux and mpegtsdemux

It is currently impossible to stream vpx codec with MPEG-TS muxer (demuxer).
This patch add this functionnality by creating a new stream type (private 0xe0) and a descriptor.

tested with vp8 and vp9
Comment 1 Tim-Philipp Müller 2017-01-06 15:05:12 UTC
That's cool. Do you happen to know where the mapping is specified?
Comment 2 Yann Jouanin 2017-01-06 15:07:19 UTC
Can you elaborate what you mean by mapping? I am not sure to understand the question.
Comment 3 Tim-Philipp Müller 2017-01-06 15:20:55 UTC
There is usually a spec or document that describes how a certain format is to be signalled inside a certain container (here: MPEG-TS). My question is: where is this described for vpx?

Also, are you aware of any sample files with this (created by others)?
Comment 4 Yann Jouanin 2017-01-06 15:24:49 UTC
From what I checked there is no proposal for vpx support in MPEG-TS.
For this reason I created my own descriptor (with the most important info).

The descriptor is inspired from A52 one (I just realised I didnt change the comments...)

        * descriptor_tag       8 uimsbf
        * descriptor_length    8 uimsbf
        * vpx_version          8 uimsbf
        * vpx_profile          8 uimsbf
        * vpx_width            16 uimsbf
        * vpx_height           16 uimsbf
        * vpx_framerate_num    16 uimsbf
        * vpx_framerate_den    16 uimsbf
Comment 5 Sebastian Dröge (slomo) 2017-01-06 19:19:02 UTC
Comment on attachment 343024 [details] [review]
Patch to add vpx support to both mpegtsmux and mpegtsdemux

That's not really anything we can merge, as it might easily conflict with other things in the future. Please work together with the WebM project or ISO to define an official MPEG-TS mapping for VP8/9/10 and then we can merge support for that.
Comment 6 GStreamer system administrator 2018-11-03 14:03:03 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/505.