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

RTCMediaStreamStats can be made obsolete #470

Closed
henbos opened this issue Aug 14, 2019 · 3 comments · Fixed by #499
Closed

RTCMediaStreamStats can be made obsolete #470

henbos opened this issue Aug 14, 2019 · 3 comments · Fixed by #499

Comments

@henbos
Copy link
Collaborator

henbos commented Aug 14, 2019

Now that things have been moved to "outbound-rtp" and "inbound-rtp", "track" stats makes less sense, and "stream" stats less so since its a list of trackIds.

We could replace the dictionary by streamIdentifiers on the "sender" and "receiver"

@henbos
Copy link
Collaborator Author

henbos commented Sep 4, 2019

  • Add streamIdentifiers to sender and receiver stats
  • Move RTCMediaStreamStats to the obsolete section

@vr000m
Copy link
Contributor

vr000m commented Sep 20, 2019

@karthikbr82 @lennart-csio would removing this impact any of our algorithms or diagnostics?

@henbos
Copy link
Collaborator Author

henbos commented Sep 20, 2019

TPAC decision: remove "track" and "stream" stats in favor of "mid"; don't need to add streamIdentifiers because that is knowable by other means and not very relevant to stats (if someone want that that should be filed separately).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants