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 599903 - Scaletempo plugin stops early when playing slowly
Scaletempo plugin stops early when playing slowly
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 0.10.17
Assigned To: Thiago Sousa Santos
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-10-28 13:00 UTC by Bastian Hecht
Modified: 2009-11-06 00:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
A short ogg/vorbis audio sample (14.39 KB, application/octet-stream)
2009-10-29 11:33 UTC, Bastian Hecht
  Details
Patch for correct calculation (113 bytes, patch)
2009-11-05 15:00 UTC, Bastian Hecht
none Details | Review
Improved patch (1.49 KB, patch)
2009-11-05 16:45 UTC, Thiago Sousa Santos
none Details | Review
Improved patch (1.14 KB, patch)
2009-11-05 17:15 UTC, Thiago Sousa Santos
none Details | Review

Description Bastian Hecht 2009-10-28 13:00:30 UTC
The scaletempo plugin stops before reaching the end of a stream for rates < 1.

To be more specific:

When playing at a rate of 0.5, the pipeline stops at about half of the stream.
I tinkered a bit around (I disabled the forwarding of eos signals) and I think that the decoder initiates the false stop.
This does not happen when playing at rates < 1 without scaletempo.

This also happens with the demo application that ships with plugins-bad: gst-plugins-bad/tests/examples/scaletempo


Built from git repository on 10/28/2009
Comment 1 Thiago Sousa Santos 2009-10-28 13:54:48 UTC
I just tried the example application with a mp3 I had around and it worked (no early stop).

What file format are you using? Can you attach a sample that fails?
Comment 2 Bastian Hecht 2009-10-29 11:33:22 UTC
Created attachment 146494 [details]
A short ogg/vorbis audio sample

espeak generated file saying "Short audio sample that fails" (more or less :)
Comment 3 Bastian Hecht 2009-10-29 11:34:25 UTC
I had no error using mp3s too.
Comment 4 Bastian Hecht 2009-11-05 14:59:05 UTC
When a new new_segment_event arrives, scaletempo doesn't calculate the stop argument correctly when forwarding it downstream.
Comment 5 Bastian Hecht 2009-11-05 15:00:38 UTC
Created attachment 147012 [details] [review]
Patch for correct calculation

558c558
<           format, start, stop, position);
---
>           format, start, stop/applied_rate, position);
Comment 6 Thiago Sousa Santos 2009-11-05 16:45:55 UTC
Created attachment 147017 [details] [review]
Improved patch

Improves the patch by updating also start and position values.
Comment 7 Thiago Sousa Santos 2009-11-05 17:15:23 UTC
Created attachment 147019 [details] [review]
Improved patch

Fixing previous patch.
Comment 8 Thiago Sousa Santos 2009-11-06 00:47:05 UTC
Fixed.

commit 892555955a0fc175768b7b684b5f00fb34496baf
Author: Thiago Santos <thiago.sousa.santos@collabora.co.uk>
Date:   Thu Nov 5 13:40:38 2009 -0300

    scaletempo: properly update new segments
    
    Scaletempo was missing an update of 'stop' in
    new segment parameters when pushing it downstream,
    which caused files to end earlier when rate < 1.
    
    Fixes #599903
    
    Based on patch by: Bastian Hecht <hechtb@gmail.com>