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 790821 - m3u8: Parse and use AVERAGE-BANDWIDTH attribute if available
m3u8: Parse and use AVERAGE-BANDWIDTH attribute if available
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 1.13.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-11-25 13:24 UTC by Hosang Lee
Modified: 2017-12-06 23:21 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
m3u8: Parse and use AVERAGE-BANDWIDTH attribute if available (2.03 KB, patch)
2017-11-25 13:27 UTC, Hosang Lee 		committed Details | Review

Description Hosang Lee 2017-11-25 13:24:18 UTC 
The AVERAGE-BANDWIDTH attribute in the EXT-X-STREAM-INF tag represents the the average segment bit rate of the Variant Stream, while the BANDWIDTH attribute represents the peak segment bit rate of the Variant Stream. (https://tools.ietf.org/html/draft-pantos-http-live-streaming-23#section-4.3.4.2)
Using the average bit rate instead of the peak bit rate for variant switching is more efficient and appropriate. Sometimes due to VBR encoding, the BANDWIDTH may represent a value way above the average bit rate, which could result to players not switching to that variant stream although network bandwidth is sufficiently available.
Comment 1 Hosang Lee 2017-11-25 13:27:23 UTC 
Created attachment 364388 [details] [review]
m3u8: Parse and use AVERAGE-BANDWIDTH attribute if available
Comment 2 Nicolas Dufresne (ndufresne) 2017-11-25 15:10:50 UTC 
Seems like a logical move.
Comment 3 Nicolas Dufresne (ndufresne) 2017-12-06 20:10:31 UTC 
Thanks, will be in the next major version of GStreamer, if you feel that a 1.12 backport would be better, let me know.

Attachment 364388 [details] pushed as 6a5bae5 - m3u8: Parse and use AVERAGE-BANDWIDTH attribute if available
Comment 4 Hosang Lee 2017-12-06 23:21:41 UTC 
Thanks.
I would appreciate if it would be in 1.12 as well.