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 606635 - Libvisual Parameters not supported
Libvisual Parameters not supported
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
unspecified
Other All
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-01-11 15:53 UTC by daniel
Modified: 2018-11-03 11:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
This enables the libvisual plugin to get/set libvisual parameters. (10.38 KB, patch)
2010-01-14 02:37 UTC, Scott Sibley
none Details | Review
This update allows dynamic property changes. (10.60 KB, patch)
2010-01-14 04:21 UTC, Scott Sibley
rejected Details | Review
This fixes the reverted patch. (10.60 KB, patch)
2010-01-18 18:28 UTC, Scott Sibley
needs-work Details | Review

Description daniel 2010-01-11 15:53:12 UTC
It would be nice if the libvisual-plugin could support setting VisParams through gStreamer as in http://libvisual.svn.sourceforge.net/viewvc/libvisual/trunk/libvisual/lv_param.h
Without it, all libvisual actors are usable with their default settings only :(
It should be pretty trivial to support the most important parameter-types:
* VISUAL_PARAM_ENTRY_TYPE_STRING
* VISUAL_PARAM_ENTRY_TYPE_INTEGER
* VISUAL_PARAM_ENTRY_TYPE_FLOAT
* VISUAL_PARAM_ENTRY_TYPE_DOUBLE
* VISUAL_PARAM_ENTRY_TYPE_COLOR

you might want to visit #libvisual on freenode for questions
Comment 1 Scott Sibley 2010-01-14 02:37:05 UTC
Created attachment 151373 [details] [review]
This enables the libvisual plugin to get/set libvisual parameters.
Comment 2 Scott Sibley 2010-01-14 02:43:03 UTC
I should have noted that since libvisual lacks annotations and such, it's impossible to include help text and property defaults. We're waiting to talk with libvisual's maintainer on a new release to include some additional features such as these.
Comment 3 Scott Sibley 2010-01-14 04:21:03 UTC
Created attachment 151379 [details] [review]
This update allows dynamic property changes.

In the above patch only initial property settings would make it through. This patch fixes that.
Comment 4 Sebastian Dröge (slomo) 2010-01-16 20:10:17 UTC
Comment on attachment 151379 [details] [review]
This update allows dynamic property changes.

This patch seems to be reverted. Could you attach it again correctly? :)

Also, any news about adding some more information to libvisual's parameters?
Comment 5 Scott Sibley 2010-01-18 18:28:40 UTC
Created attachment 151690 [details] [review]
This fixes the reverted patch.
Comment 6 Scott Sibley 2010-01-18 18:31:29 UTC
Sebastian, there's some work on libvisual's svn. Annotations are in at the moment, plus defaults and min/max, so basically everything to satisfy gstreamer.
Comment 7 Sebastian Dröge (slomo) 2011-05-30 10:45:25 UTC
Comment on attachment 151690 [details] [review]
This fixes the reverted patch.

Please update the patch to use the libvisual annotations for properties. Without it's not very useful imho
Comment 8 GStreamer system administrator 2018-11-03 11:16:10 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-base/issues/28.