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

Rtt variance #69

Closed
wants to merge 2 commits into from
Closed

Conversation

JohannesDeml
Copy link
Contributor

  • Add an interface for peer->roundTripTimeVariance to get information of the current RTT variance
  • Add information to the documentation of how the RTT and its variance are calculated

@nxrighthere
Copy link
Owner

This data is not intended for usage by users.

@nxrighthere nxrighthere closed this Mar 4, 2021
@JohannesDeml
Copy link
Contributor Author

JohannesDeml commented Mar 4, 2021

Not sure if I agree with you on (the variance can be used to get a quick idea of the current packet jitter), but I do appreciate that you want to keep the interface as tidy as possible.

@nxrighthere
Copy link
Owner

the variance can be used to get a quick idea of the current packet jitter

The algorithm behind it is suboptimal for that, it's primarily designed for protocol's congestion control. I don't see how it can be useful for a user.

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