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 537700 - [scaletempo] Scale audio tempo in sync with playback rate
[scaletempo] Scale audio tempo in sync with playback rate
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
0.3.3
Other Linux
: Normal enhancement
: 0.10.12
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-06-10 23:47 UTC by Rov Juvano
Modified: 2009-05-07 11:13 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
scaletempo patch against -bad-cvs (31.54 KB, patch)
2008-06-16 11:54 UTC, Rov Juvano
committed Details | Review
Patch to fix gconfaudiosink typo in demo-player.c (353 bytes, patch)
2008-09-07 23:14 UTC, Rov Juvano
committed Details | Review

Description Rov Juvano 2008-06-10 23:47:54 UTC
This new plugin maintains audio pitch when playback changes (i.e. no more chipmunk effect).

The code along with a demo app is available at:
http://sourceforge.net/project/showfiles.php?group_id=220192

with a git repos at:
http://repo.or.cz/w/gst-scaletempo-rj.git
http://repo.or.cz/w/gst-scaletempo-demo-rj.git
Comment 1 David Schleef 2008-06-11 00:43:30 UTC
The repositories are not accessible.

Patches to gst-plugins-bad would be better.  Put your plugin code in gst-plugins-bad/gst/scaletempo.

(First comment: I'm not convinced that scaletempo is the best name for this plugin.  The name doesn't really tell me what it does.)
Comment 2 Rov Juvano 2008-06-16 11:54:13 UTC
Created attachment 112829 [details] [review]
scaletempo patch against -bad-cvs
Comment 3 Rov Juvano 2008-07-08 06:15:30 UTC
Has anyone had a chance to look at this?
Comment 4 Sebastian Dröge (slomo) 2008-07-08 07:05:14 UTC
Not yet... looking at it now.

Do you have an idea for a better name than scaletempo? :)
Comment 5 Rov Juvano 2008-07-08 12:33:42 UTC
Unfortunately, that is the name it has come to be known by outside of GStreamer.
Comment 6 Sebastian Dröge (slomo) 2008-07-08 12:57:16 UTC
Ok, then let's keep that name :)

I just took a short look until now but one thing I noticed is that you use instance private data... you could as well put all the private data into the instance struct instead. This will make things a bit faster.

Apart from that I didn't notice anything yet, will look closer later :)
Comment 7 Sebastian Dröge (slomo) 2008-08-31 12:19:39 UTC
2008-08-31  Sebastian Dröge  <sebastian.droege@collabora.co.uk>

	Patch by: Rov Juvano <rovjuvano at users dot sourceforge dot net>

	* configure.ac:
	* docs/plugins/Makefile.am:
	* docs/plugins/gst-plugins-bad-plugins-docs.sgml:
	* docs/plugins/gst-plugins-bad-plugins-sections.txt:
	* docs/plugins/inspect/plugin-scaletempo.xml:
	* examples/scaletempo/Makefile.am:
	* examples/scaletempo/demo-gui.c: (pop_status_bar),
	(status_bar_printf), (demo_gui_seek_bar_format), (update_position),
	(demo_gui_seek_bar_change), (demo_gui_do_change_rate),
	(demo_gui_do_set_rate), (demo_gui_do_rate_entered),
	(demo_gui_do_toggle_advanced), (demo_gui_do_toggle_disabled),
	(demo_gui_do_seek), (demo_gui_do_play), (demo_gui_do_pause),
	(demo_gui_do_play_pause), (demo_gui_do_open_file),
	(demo_gui_do_playlist_prev), (demo_gui_do_playlist_next),
	(demo_gui_do_about_dialog), (demo_gui_do_quit),
	(demo_gui_request_set_stride), (demo_gui_request_set_overlap),
	(demo_gui_request_set_search), (demo_gui_rate_changed),
	(demo_gui_playing_started), (demo_gui_playing_paused),
	(demo_gui_playing_ended), (demo_gui_player_errored),
	(demo_gui_stride_changed), (demo_gui_overlap_changed),
	(demo_gui_search_changed), (demo_gui_set_player_func),
	(demo_gui_set_playlist_func), (build_gvalue_array),
	(create_action), (demo_gui_show_func), (demo_gui_set_player),
	(demo_gui_set_playlist), (demo_gui_show), (demo_gui_get_property),
	(demo_gui_set_property), (demo_gui_init), (demo_gui_class_init),
	(demo_gui_get_type):
	* examples/scaletempo/demo-gui.h:
	* examples/scaletempo/demo-main.c: (handle_error_message),
	(handle_quit), (main):
	* examples/scaletempo/demo-player.c: (no_pipeline),
	(demo_player_event_listener), (demo_player_state_changed_cb),
	(demo_player_eos_cb), (demo_player_build_pipeline), (_set_rate),
	(demo_player_scale_rate_func), (demo_player_set_rate_func),
	(_set_state_and_wait), (demo_player_load_uri_func),
	(demo_player_play_func), (demo_player_pause_func), (_seek_to),
	(demo_player_seek_by_func), (demo_player_seek_to_func),
	(demo_player_get_position_func), (demo_player_get_duration_func),
	(demo_player_scale_rate), (demo_player_set_rate),
	(demo_player_load_uri), (demo_player_play), (demo_player_pause),
	(demo_player_seek_by), (demo_player_seek_to),
	(demo_player_get_position), (demo_player_get_duration),
	(demo_player_get_property), (demo_player_set_property),
	(demo_player_init), (demo_player_class_init),
	(demo_player_get_type):
	* examples/scaletempo/demo-player.h:
	* gst/scaletempo/Makefile.am:
	* gst/scaletempo/gstscaletempo.c: (best_overlap_offset_float),
	(best_overlap_offset_s16), (output_overlap_float),
	(output_overlap_s16), (fill_queue), (reinit_buffers),
	(gst_scaletempo_transform), (gst_scaletempo_transform_size),
	(gst_scaletempo_sink_event), (gst_scaletempo_set_caps),
	(gst_scaletempo_get_property), (gst_scaletempo_set_property),
	(gst_scaletempo_base_init), (gst_scaletempo_class_init),
	(gst_scaletempo_init):
	* gst/scaletempo/gstscaletempo.h:
	* gst/scaletempo/gstscaletempoplugin.c: (plugin_init):
	Add scaletempo plugin, which allows to scale the speed of audio without
	changing the pitch by handling seeks with a rate!=1.0.
	Integrate it into the docs and add the example application for it.
	Fixes bug #537700.
Comment 8 Rov Juvano 2008-09-07 23:14:59 UTC
Created attachment 118263 [details] [review]
Patch to fix gconfaudiosink typo in demo-player.c

This patch fixes a typo in demo-player.c that made gconfaudiosink required instead of autodetected.
Comment 9 Sebastian Dröge (slomo) 2009-05-07 11:13:47 UTC
commit d32c04c0040d944b32305692a1aaa1f3e5568a9e
Author: Rov Juvano <rovjuvano@users.sourceforge.net>
Date:   Thu May 7 13:12:34 2009 +0200

    scaletempo: Don't require gconfaudiosink in the demo
    
    Fixes bug #537700.