Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Further approach for measurement and estimation of throughput calculation in LL CTE streaming #3660

Conversation

mlasak
Copy link
Contributor

@mlasak mlasak commented Jun 9, 2021

Adding a further approach for measurement and estimation (FAME) for chunked transfer encoding based streaming.

Implemented concepts:

  • Make use of availabilityStartTime to decide if segment is downloaded at network speed or at producer rate
  • Make use of availabilityStartTime to decide which chunks are downloaded at network speed or at producer rate
  • Keep track of the timings of recent chunk download timings (last segments)

This fixes, e.g.
#3538

This PR adds a new kind of needed test: unit tests in browser scope. This is necessary as some API (e.g. fetch API) is not available in node.js context.

  • add config parameter to reference player
  • further tests, all possible patterns
  • more documentation (update: added sample page)
  • decide if concept of artificial delays (to enforce some chunks to be downloaded at network speed) should be added (update: discussed approach during monthly call)

@mlasak mlasak requested a review from dsilhavy June 9, 2021 06:36
@mlasak mlasak changed the title WIP: Feat ll new tp calc approach WIP: Further approach for measurement and estimation of throughput calculation in LL CTE streaming Jun 9, 2021
@dsilhavy dsilhavy added this to the 4.0.0 milestone Jun 9, 2021
Adds as well new kind of tests, unit tests in browser env with karma
still todo:
- tests
- finalize artificial delay approach
also removing some irrelevant log outputs
@mlasak mlasak force-pushed the feat-ll-new-tp-calc-approach branch from f050883 to d12079e Compare June 14, 2021 14:01
@mlasak mlasak changed the title WIP: Further approach for measurement and estimation of throughput calculation in LL CTE streaming Further approach for measurement and estimation of throughput calculation in LL CTE streaming Jun 15, 2021
@dsilhavy dsilhavy merged commit ea5442c into Dash-Industry-Forum:development Jun 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants