libp2p / libp2p/test-plans

perf: latency dashboard is misleading

Open
#277 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
65
Forks
82
PR merge metrics
No merged PRs in 30d

Description

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.

Contributor guide

No contributing guide indexed for this repository

Research direction

The issue does not name files or tests; start by locating the latency dashboard and the code that combines connection-establishment measurements with ping results. Compare the plotted metrics and evaluate the proposed round-trip normalization. Done means the visualization no longer makes other implementations look worse because of the extra round trip.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data-visualization, networking, performance
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.