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 731921 - gl: Port glmixer to the GstVideoAggregator baseclass
gl: Port glmixer to the GstVideoAggregator baseclass
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other All
: Normal enhancement
: 1.3.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 731919
Blocks:
 
 
Reported: 2014-06-19 15:40 UTC by Thibault Saunier
Modified: 2014-06-25 00:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gl: Port glmixer to the GstVideoAggregator baseclass (76.62 KB, patch)
2014-06-19 15:40 UTC, Thibault Saunier
committed Details | Review

Description Thibault Saunier 2014-06-19 15:40:17 UTC
The new GstVideoAggregator base class feets perfectly for the glmixer  base class and
removes quite a lot of code.
Comment 1 Thibault Saunier 2014-06-19 15:40:20 UTC
Created attachment 278778 [details] [review]
gl: Port glmixer to the GstVideoAggregator baseclass
Comment 2 Thibault Saunier 2014-06-19 16:10:02 UTC
You can find a branch with all the related commits at: https://gitlab.com/thiblahute/gst-plugins-bad/commits/aggregator
Comment 3 Thibault Saunier 2014-06-20 18:12:21 UTC
commit 4ce8b409cc30d80c5d6ce949965789036e2269d6
Author: Thibault Saunier <tsaunier@gnome.org>
Date:   Tue Jun 10 11:26:53 2014 +0200

    gl: Port glmixer to the GstVideoAggregator baseclass
    
    https://bugzilla.gnome.org/show_bug.cgi?id=731921
Comment 4 Robert Swain 2014-06-20 22:11:19 UTC
This appears to be non-functional for me with git master:

gst-launch-1.0 mix. ! queue ! glimagesink \
gltestsrc pattern=black ! video/x-raw,width=1280,height=720 ! queue name=background \
gltestsrc pattern=smpte ! glfiltercube ! video/x-raw,width=1280,height=720 ! glcolorscale ! video/x-raw,width=640,height=720 ! queue name=left \
gltestsrc pattern=smpte ! glfilterglass ! video/x-raw,width=1280,height=720 ! glcolorscale ! video/x-raw,width=640,height=720 ! queue name=right \
background. ! mix.sink_0 \
left. ! mix.sink_1 \
right. ! mix.sink_2 \
glvideomixer name=mix sink_2::xpos=640

(gst-launch-1.0:65455): GStreamer-CRITICAL **: Padname sink_4294967295 is not unique in element mix, not adding

(gst-launch-1.0:65455): GStreamer-CRITICAL **: Padname sink_4294967295 is not unique in element mix, not adding
WARNING: erroneous pipeline: could not link left to mix

The same pipeline works fine if I just swap out glvideomixer for videomixer.

Do you think this is related or should it be reported as a separate issue? It would be nice to have glvideomixer functioning properly for 1.4.0 I think. :)
Comment 5 Thibault Saunier 2014-06-21 12:24:19 UTC
(In reply to comment #4)
> This appears to be non-functional for me with git master:
> 
> gst-launch-1.0 mix. ! queue ! glimagesink \
> gltestsrc pattern=black ! video/x-raw,width=1280,height=720 ! queue
> name=background \
> gltestsrc pattern=smpte ! glfiltercube ! video/x-raw,width=1280,height=720 !
> glcolorscale ! video/x-raw,width=640,height=720 ! queue name=left \
> gltestsrc pattern=smpte ! glfilterglass ! video/x-raw,width=1280,height=720 !
> glcolorscale ! video/x-raw,width=640,height=720 ! queue name=right \
> background. ! mix.sink_0 \
> left. ! mix.sink_1 \
> right. ! mix.sink_2 \
> glvideomixer name=mix sink_2::xpos=640
> 
> (gst-launch-1.0:65455): GStreamer-CRITICAL **: Padname sink_4294967295 is not
> unique in element mix, not adding
> 
> (gst-launch-1.0:65455): GStreamer-CRITICAL **: Padname sink_4294967295 is not
> unique in element mix, not adding
> WARNING: erroneous pipeline: could not link left to mix
> 
> The same pipeline works fine if I just swap out glvideomixer for videomixer.
> 
> Do you think this is related or should it be reported as a separate issue? It
> would be nice to have glvideomixer functioning properly for 1.4.0 I think. :)

This has been fixed with:

commit 6409641130e008c7c48c5f97da34e1d21a8f1698
Author: Thibault Saunier <tsaunier@gnome.org>
Date:   Sat Jun 21 13:45:13 2014 +0200

    aggregator: Fix requested pad name


Pretty nice pipeline btw :)
Comment 6 Robert Swain 2014-06-21 13:03:48 UTC
Right, that fixed it for gltestsrc. I'll file a new ticket for avfvideosrc.