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

M2: Ambiguities in the standard #189

Open
mcamardon opened this issue Apr 26, 2023 · 2 comments
Open

M2: Ambiguities in the standard #189

mcamardon opened this issue Apr 26, 2023 · 2 comments
Assignees
Labels
question Further information is requested

Comments

@mcamardon
Copy link
Contributor

mcamardon commented Apr 26, 2023

We have some keys (dl, bl or mtp) that are reported with a value equal to zero. We discoverd this in videojs and dash players.

AC:

  1. Investigate if the value is zero or is something wrong with the player
  2. Ask to CMCD's team if this has sense.
@mcamardon mcamardon converted this from a draft issue Apr 26, 2023
@mcamardon mcamardon added the question Further information is requested label Apr 26, 2023
@mcamardon mcamardon added this to the M2: A simple web validator milestone Apr 26, 2023
@mcamardon mcamardon self-assigned this Apr 26, 2023
@mcamardon mcamardon moved this from 🔖 Ready to start to 🏗 In progress in CMCD Validator & Analyzer Apr 26, 2023
@mcamardon mcamardon moved this from 🏗 In progress to 👀 In review in CMCD Validator & Analyzer Apr 26, 2023
@mcamardon
Copy link
Contributor Author

After some investigation, I created an issue in the cmcd repository asking for clarification on this. The issue is cta-wave/common-media-client-data#100.

@mcamardon mcamardon moved this from 👀 In review to 🏗 In progress in CMCD Validator & Analyzer Apr 26, 2023
@CamiloBorba09 CamiloBorba09 moved this from 🏗 In progress to 👀 In review in CMCD Validator & Analyzer Apr 28, 2023
@mcamardon
Copy link
Contributor Author

mcamardon commented May 4, 2023

We got a response to the issue we created on the cmcd repository. I'll leave a brief summary for each key:

  • bl: It is possible to sent it with 0. However it is incorrect to use a value of 0 if the buffer length is unknown.
  • mtp: It should not be possible to send it as 0. A value of 0 does not make sense since it should report what what the player measured and any measurement is going to have some non-zero value associated with it.
  • dl: A deadline of 0 would mean that you need the object instantly (versus an infinite waiting time). It is legal to send a value of 0, however this is impossible for the server to fulfil and it would likely ignore that key as a result.
  • rtp: It can be sent with a value of 0. This would be self-destructive for the player and it would be silly to send it, but it is legal. The server would likely consider it nonsensical and ignore it.

@mcamardon mcamardon moved this from 👀 In review to ✅ Done in CMCD Validator & Analyzer May 4, 2023
@JoaquinBCh JoaquinBCh moved this from ✅ Done to ✅ Done old in CMCD Validator & Analyzer May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
Status: Done old
Development

No branches or pull requests

1 participant