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

perf: latency dashboard is misleading #277

Open
marten-seemann opened this issue Aug 25, 2023 · 1 comment
Open

perf: latency dashboard is misleading #277

marten-seemann opened this issue Aug 25, 2023 · 1 comment

Comments

@marten-seemann
Copy link
Contributor

The latency dashboard shows connection establishment latency + 1 round trip, and shows the ping latency in the same plot. ICMP Ping doesn't contain the additional 1 round trip, so it makes all other implementations look worse than they actually are.

Not ideal, but maybe a better way to visualize things is to plot measurement results in multiples of the round trip, and subtract the 1 round trip from our measurement results.

@mxinden
Copy link
Member

mxinden commented Aug 25, 2023

I see your point.

I am hesitant to do any math beyond a simple boxplot on the data, as that would obfuscate details.

Instead I suggest to tackle the confusion with better documentation. We already have the line below. I think for each graph we should put the baseline (ping and iperf) into context.

show baseline - whether to display the baseline measurement, that is ping for latency and iperf for throughput:

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

No branches or pull requests

2 participants