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 679031 - playbin/playsink: Add support for audio and video filters
playbin/playsink: Add support for audio and video filters
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal enhancement
: 1.3.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
playback
: 696045 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-06-28 09:27 UTC by Sebastian Dröge (slomo)
Modified: 2014-03-16 17:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
playsink: Fix documentation for what the audio chain looks like (1.83 KB, patch)
2014-03-15 15:11 UTC, Arun Raghavan
committed Details | Review
playback: Add video-/audio-filter properties (24.53 KB, patch)
2014-03-15 15:12 UTC, Arun Raghavan
none Details | Review
playback: Add video-/audio-filter properties (25.88 KB, patch)
2014-03-16 16:01 UTC, Arun Raghavan
committed Details | Review

Description Sebastian Dröge (slomo) 2012-06-28 09:27:16 UTC
Could be an additional property audio-filter and video-filter that playbin then inserts in front of the actual sink. This is useful for adding things like an equalizer or some video postprocessing elements in front of the sink.

Currently applications use a custom sink-bin for this but this has the disadvantage that playbin's internal sink selection mechanism is not used.
Comment 1 Bastien Nocera 2013-06-14 13:33:16 UTC
See also bug 696045, which asks for an audio filter property that could be bypassed when passthrough is used. The fact that it's bypassed should be advertised to the application, so it doesn't offer the functionality of the filter (in totem's case, it would reset the playback rate to 1x).
Comment 2 sreerenj 2013-06-14 14:32:30 UTC
Another one related to this: https://bugzilla.gnome.org/show_bug.cgi?id=687182
Comment 3 Sebastian Dröge (slomo) 2013-08-16 10:02:47 UTC
*** Bug 696045 has been marked as a duplicate of this bug. ***
Comment 4 Arun Raghavan 2014-03-15 15:11:48 UTC
Created attachment 272016 [details] [review]
playsink: Fix documentation for what the audio chain looks like
Comment 5 Arun Raghavan 2014-03-15 15:12:00 UTC
Created attachment 272017 [details] [review]
playback: Add video-/audio-filter properties

This provides an audio-filter and video-filter property to allow
applications to set filter elements/bins. The idea is that these will be
applied if possible -- for non-raw sinks, the filters will be skipped.

If the application wishes to force the application of the filters, this
can be done by setting the new flag introduced on playsink -
GST_PLAY_FLAG_FORCE_FILTERS.
Comment 6 Arun Raghavan 2014-03-15 15:19:18 UTC
(In reply to comment #5)
> Created an attachment (id=272017) [details] [review]
> playback: Add video-/audio-filter properties

The string equivalent for the GST_PLAY_FLAG_FORCE_FILTERS is missing. I've added that locally, and appended the flag to the copy of the enum in b/tests/examples/playback/playback-test.c. Will update the patch here based on any other comments that come up.
Comment 7 Arun Raghavan 2014-03-16 16:01:42 UTC
Created attachment 272068 [details] [review]
playback: Add video-/audio-filter properties

This provides an audio-filter and video-filter property to allow
applications to set filter elements/bins. The idea is that these will
e
applied if possible -- for non-raw sinks, the filters will be skipped.

If the application wishes to force the application of the filters, this
can be done by setting the new flag introduced on playsink -
GST_PLAY_FLAG_FORCE_FILTERS.
Comment 8 Sebastian Dröge (slomo) 2014-03-16 17:39:08 UTC
commit f4cab18ec13548426245e5cc1ae43caa69a42317
Author: Arun Raghavan <arun@accosted.net>
Date:   Sun Mar 16 17:00:38 2014 +0100

    playback: Add video-/audio-filter properties
    
    This provides an audio-filter and video-filter property to allow
    applications to set filter elements/bins. The idea is that these will
    e
    applied if possible -- for non-raw sinks, the filters will be skipped.
    
    If the application wishes to force the application of the filters, this
    can be done by setting the new flag introduced on playsink -
    GST_PLAY_FLAG_FORCE_FILTERS.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=679031