I've completed re-writing GstBaseSrc to implement 2 src pads within

[caoxi]

Hey guys, I've completed my work on re-writing GstBaseSrc(and also GstPushSrc) class, in order to allow more source pads to output video data. please check flowchart above. In this case, I didn't use 'tee' element to do stream copy or any other element to mux the streams.
The whole pipeline is:
gst-launch-1.0 my-src-plugin name=t t.src ! queue ! video/x-raw,format=YUY2,width=1280,height=720 ! videoconvert ! ximagesink t.video ! queue ! video/x-raw,format=YUY2,width=1920,height=1080 ! videoconvert ! ximagesink

this pipline will pop 2 preview windows with configured resolutions/formats on screen.

This shows that GstBaseSrc is able to implement multi src pads withouth using any other elements

I'll post more details about my design afterwards. Thanks!

[caoxi]

BTW, this flowchart is generated by using 'dot' tools

[Sebastian Dröge-3]

On Wed, 2017-04-12 at 19:29 -0700, caoxi wrote:

> <http://gstreamer-devel.966125.n4.nabble.com/file/n4682641/ms_pipeline.png> 
>
> Hey guys, I've completed my work on re-writing GstBaseSrc(and also
> GstPushSrc) class, in order to allow more source pads to output video data.
> please check flowchart above. In this case, I didn't use 'tee' element to do
> stream copy or any other element to mux the streams.
> The whole pipeline is:
> gst-launch-1.0 my-src-plugin name=t t.src ! queue !
> video/x-raw,format=YUY2,width=1280,height=720 ! videoconvert ! ximagesink
> t.video ! queue ! video/x-raw,format=YUY2,width=1920,height=1080 !
> videoconvert ! ximagesink
>
> this pipline will pop 2 preview windows with configured resolutions/formats
> on screen.
>
> This shows that GstBaseSrc is able to implement multi src pads withouth
> using any other elements
>
> I'll post more details about my design afterwards. Thanks!

That sounds great. Could you put this as a patch into Bugzilla for
further discussion?
  https://bugzilla.gnome.org/enter_bug.cgi?product=GStreamer

As this probably breaks API and ABI of basesrc/pushsrc, it probably
makes sense to call this GstBaseSrc2/GstPushSrc2 but they should
probably still be in gstreamer/libs/gst/base.


--
Sebastian Dröge, Centricular Ltd · http://www.centricular.com
_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[caoxi]

yes, the inherit tree is changed according to my design

[caoxi]

How can I raise a proposal to ask Gst official developers to review my changes??????
Can anyone gives me a hint? Thanks a lot

[Sebastian Dröge-3]

On Tue, 2017-04-18 at 01:21 -0700, caoxi wrote:
> How can I raise a proposal to ask Gst official developers to review
> my changes??????
> Can anyone gives me a hint? Thanks a lot 

File a bug with a "git format-patch" formatted patch against
gst-plugins-bad here:
  https://bugzilla.gnome.org/enter_bug.cgi?product=GStreamer

We're using Bugzilla for code review.

--
Sebastian Dröge, Centricular Ltd · http://www.centricular.com
_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

[caoxi]

Thanks!