nginx-ts-module icon indicating copy to clipboard operation
nginx-ts-module copied to clipboard

Published 20 hours ago • verified publisher icon arut

Updated timeShiftBufferDepth to be up to spec

Open Wassup789 opened this issue 8 years ago • 2 comments

As specified in MPEG-DASH spec, ISO 23009-1:2014, the @timeShiftBufferDepth element value should represent the time guaranteed to be available for viewing. This also enables VOD features as well if set properly.

This commit changes the buffer depth to the maximum number of segments multiplied by the minimum buffer time of one segment since that length is guaranteed to exist.

Wassup789 avatar Sep 01 '17 02:09 Wassup789

@Wassup789 I did some research and came up with the current formula based on how dash.js treats the values. I will try your approach too. Thanks for your work.

arut avatar Nov 30 '17 20:11 arut

@Wassup789 , @arut , Hi 👋.

this pull request seems has a typo.

https://github.com/nginx-with-docker/nginx-ts-module-src/commit/436a73929d125b6383316b3a2a8db8ec3e830287

I've make a pre-built module docker images for this nginx module project, you can try here: https://github.com/nginx-with-docker/ngx_http_ts_module

soulteary avatar Jun 15 '21 09:06 soulteary