search

w3c / webrtc-provisional-stats

https://w3c.github.io/webrtc-provisional-stats/
Other
7 stars 11 forks source link

Add RTCMediaStreamTrackStats.concealedAudibleSamples #2

Open henbos opened 6 years ago

henbos commented 6 years ago

The PR on the webrtc-stats spec did not make it because we could not come up with a definition of what it means for samples to be "audible" that is clear or has a citable reference, see https://github.com/w3c/webrtc-stats/pull/215.

Provisional definition:

Only present for inbound audio tracks. The total number of concealed audio samples (see concealedSamples) that was played out during an audible portions of the stream. Audible means that the received audio is not considered background noise or silence by the user agent. It is up to the implementation to determine what is considered background noise, but concealments of audible samples SHOULD in general have a greater impact on user experience than concealment of non-audible samples. If the voice activity flag is present in RTP packets as per [[RFC6464]] this MAY be used to indicate audibility. Audibility MAY also be based on audio levels or more sophisticated analysis of the stream.

In the meantime, we can land this here.

henbos commented 6 years ago

Will make a PR as soon as webrtc-stats has merged changes splitting the RTCMediaStreamTrackStats into multiple dictionaries (local/remote + audio/video) https://github.com/w3c/webrtc-stats/pull/272 and RTCRtpSender/ReceiverStats https://github.com/w3c/webrtc-stats/pull/273.