[Bug 747558] gsthlsdemux.c/gstadaptivedemux.c: Possible Improvement: Variant selection doesn't consider hysteresis

[GStreamer (GNOME Bugzilla)]

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

Duncan Palmer <dpalmer at digisoft.tv> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dpalmer at digisoft.tv

--- Comment #6 from Duncan Palmer <dpalmer at digisoft.tv> ---
I'm revisiting this for master... Our solution is currently specific to
hlsdemux, and I'd like to change it so it can be used in adaptivedemux instead
of requiring 3 separate implementations.

So, I was thinking along the following lines;
- Restore the bitrate averaging logic, which was (I think) mistakenly removed
as part of https://bugzilla.gnome.org/show_bug.cgi?id=733959
- Apply a noise filter to the resulting bitrate; if the delta between the newly
calculated bitrate and the previous is < some threshold (50 kpbs say), then
keep the old bitrate.

Our previous solution applies hysterisis when a variant change is in the offing
due to the new bitrate. This is why it's hls specific. However, simply applying
a noise filter should achieve the same thing I believe.

Does anyone have any comments on this approach?

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