docs(beta): NAT gap now carries the lab numbers - #86
Conversation
Netsim (#82, #84) measured the matrix; the gap that remains is real-network data, so say that instead. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KeiotDVE94wEzvc7wcvm1y
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 17dc9f05a3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ([netsim/results/results.md](netsim/results/results.md)) shows attach in | ||
| 0.2–1.3 s and resume after a network flip in ~2.6 s direct / ~3.0 s over |
There was a problem hiding this comment.
The linked matrix does not show all attach measurements within 0.2–1.3 s: its flip-turn row records a 2,289 ms maximum. The stated range only matches the scenario p50 values after rounding, so presenting it as the unqualified attach range understates the observed latency; either identify these as p50s or extend the range to include the measured maximum.
Useful? React with 👍 / 👎.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
1 participant
Netsim measured the matrix (#82, #84); the remaining gap is real-network data.
🤖 Generated with Claude Code
https://claude.ai/code/session_01KeiotDVE94wEzvc7wcvm1y
Note
Low Risk
Documentation-only change to BETA.md with no runtime or security impact.
Overview
Updates the beta Known gaps section so reconnect/NAT coverage is described more accurately.
The old NAT and network numbers bullet (tuned timing, no published measurements) is replaced with Real-network numbers, which points readers to the Docker NAT matrix in
netsim/results/results.mdand cites lab attach (~0.2–1.3 s) and post-flip resume (~2.6 s direct / ~3.0 s over TURN). The gap is now framed as missing published data from real home, office, and cellular networks—not missing lab work.Reviewed by Cursor Bugbot for commit 17dc9f0. Bugbot is set up for automated code reviews on this repo. Configure here.