[Bug 787358] debugutils: Added new jitterer element

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=787358

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slomo at coaxion.net

--- Comment #7 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
(In reply to Sebastian Dröge (slomo) from comment #5)

> ::: gst/debugutils/gstjitterer.c
> @@ +119,3 @@
> +          "Average of the drift to apply",
> +          -G_MAXINT64 / 2, G_MAXINT64 / 2, 0,
> +          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
> 
> This is drift per buffer, which seems a bit difficult to control. Better
> would be something like drift per second, but not sure how to implement that
> easily just from the buffer timestamps. One could calculate a running
> average of buffer durations (as calculated from timestamps) probably and
> base it on that.

Even easier actually. No drift on first buffer, drift on second buffer is
scaled by the timestamp diff between current and previous buffer, drift on
third by current and previous again, etc. Just need to remember the last buffer
timestamp for this.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.