-
Notifications
You must be signed in to change notification settings - Fork 19
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
Use abs-capture-time header extension instead of RTCP SR for calculating captureTime on remote sources when available #86
Comments
@drkron, any thoughts? I don't have much WebRTC experience. |
pinging the usual suspects @fippo @henbos @alvestrand @jan-ivar @aboba |
In practice, captureTime is only available via RVFC for locally captured frames. Are you looking to obtain it on the remote peer as well? |
captureTime is already supported for remote webrtc sources
However relying on RTCP RR or rtrr doesn't provide insightfull information on an SFU scenario. Using the abs-capture-time value would be thr best in this case. |
When it comes to the "remote" part of a) RFC 3550 Section 6.4.1 provides the sender with RTT estimations but what we need is RTT estimations at the receiver. This means that the receiver must either also send its own RTCP Note that even if the receiver does send its own b) As @murillo128 mentioned above, RFC 3550 Section 6.4.1 and its derivatives are unable to "look beyond" RTCP-terminating mixers. I believe that it would be more useful to redefine For example, changing the text from:
To say something along the lines of:
In an ideal world,
This is basically |
The neat thing (when it works) with the current definition is that all timesstamps are using the same reference and can be compared to performance.now(). This makes it very simple to calculate glass-to-glass delay, receive-to-render delay, etc. I would suggest that absoluteCaptureTime is added next to the capture timestamp. This timestamp would then be the unaltered capture timestamp in the sender's NTP clock. |
Current approach to use the RTCP SR synchronization timestamp for the captureTime has two flaws:
In webrtc we already have a working solution which will allow to support it on both cases:
https://w3c.github.io/webrtc-extensions/#dom-rtcrtpcontributingsource-capturetimestamp
Should we include that if the abs-capture-time header extension is available we should use it instead of the RTCP SR value?
The text was updated successfully, but these errors were encountered: