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 549028 - Tempo/Pitch plugin
Tempo/Pitch plugin
Status: RESOLVED DUPLICATE of bug 417141
Product: totem
Classification: Core
Component: Movie player
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2008-08-22 16:17 UTC by Bastien Nocera
Modified: 2012-04-18 17:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add a pitch/tempo plugin (22.06 KB, patch)
2008-09-20 21:39 UTC, Philip Withnall
needs-work Details | Review
Graph of the modified GStreamer pipeline. (319.12 KB, image/png)
2008-09-20 21:40 UTC, Philip Withnall
  Details

Description Bastien Nocera 2008-08-22 16:17:11 UTC
As mentioned by Mark
http://blog.markvdb.be/2008/01/oh-flying-spaghetti-monster-music.html

Would be nice to implement this as a plugin.

The GStreamer backend would need to insert the tempo plugin, the xine-lib should use the XINE_PARAM_FINE_SPEED parameter to set the playback speed.

You'll need to add API to the bacon-video-widget:
void bacon_video_widget_set_tempo (BaconVideoWidget *bvw, float tempo);

Default would be 1.0.
Comment 1 Philip Withnall 2008-09-20 21:39:16 UTC
Created attachment 119054 [details] [review]
Add a pitch/tempo plugin

Here's a first go at it. There's an interface for changing the pitch/tempo, and the pitch element is inserted into the pipeline nicely. Unfortunately, the video and audio don't stay in sync when the tempo's changed --- the video proceeds at normal pace, regardless of the audio's tempo.
Comment 2 Philip Withnall 2008-09-20 21:40:06 UTC
Created attachment 119055 [details]
Graph of the modified GStreamer pipeline.

Here's a graph of the GStreamer pipeline after the pitch element is inserted.
Comment 3 Philip Withnall 2008-09-20 21:41:04 UTC
Tim, any ideas on how to keep the audio and video in sync?
Comment 4 Philip Withnall 2008-10-07 19:08:35 UTC
I forgot to mention that the patch doesn't implement the new BVW functions in the xine backend yet.
Comment 5 Bastien Nocera 2009-04-12 01:19:00 UTC
See also:
http://thread.gmane.org/gmane.comp.gnome.apps.rhythmbox.devel/9344

FWIW, we shouldn't be blocking on implementing the xine backend bits.
Comment 6 Bastien Nocera 2012-01-06 15:59:31 UTC
See also:
http://article.gmane.org/gmane.comp.gnome.apps.rhythmbox.devel/11936/
Comment 7 Bastien Nocera 2012-04-18 17:04:09 UTC
Bug 417141 contains a patch for the backend (now merged), and a patch to implement the UI.

*** This bug has been marked as a duplicate of bug 417141 ***