GNOME Bugzilla – Bug 696135
h264parse: Add mvc stream parsing support
Last modified: 2015-01-20 13:37:57 UTC
Created attachment 239246 [details] [review] videoparsers: h264parse: Add MVC stream parsing support. Hi, This is to add h264 mvc stream parsing support to h264parse element based on the codecparser API provided in https://bugzilla.gnome.org/show_bug.cgi?id=685215. Attaching the initial patch which handles the following parts, *parse subset sequence parameter set *taking care of slice_nal_extenion *taking care of MVCDecoderConfigurationRecord. *and other necessary stuffs This is still missing the Stereo_Video_Info and Frame_Packing_Arrangement SEI payload parsing stuffs which has been added to codec_parser library (bug: 685215). This one will be added based on https://bugzilla.gnome.org/show_bug.cgi?id=611157 . As per the comment2 in bug:611157 from Stefan Saur, "The caps would tell whether it is a multichannel videostream (channels={1,2}) and in case of 2 how the frames are packed(channel-layout={mono,separate,stereo-interleaved,stereo-over-under,stereo-left-right})." Expecting some comments.. ?
Hi, the code for detecting AU boundaries needs to be updated too, I think. Only checking for first_mb_in_slice == 0 doesn't look correct in the general case, even for standard AVC cases (baseline profile with ASO, in particular). This also reminds me to create a new entry for the demuxers and/or link it to here. The most important thing there would be to handle "stream-id" and "layer-id" (for SVC) properties. Otherwise, the current behaviour is that we generate as many decoder + sinks as there are substreams IIRC.
Created attachment 279221 [details] [review] h264parse: add initial support for MVC NAL units Same patch as sree's but without the MVC configuration record from MP4 spec. Additional fixes for robustness and optimization purposes.
ping? Any committer could review this patch? I think it is a good idea to have it to support multiview and stereo profiles in the future.
Jan was working on integrating that.
Yes, I should have pushed this a while ago