Repository navigation
Increment C is built, and the docs stop saying nothing is live - #72
Merged
Merged
Conversation
On 2026-09-02 pgb read `?format=bands` payloads off the live server and drew from them. That closed increment C on the wire, and the server stayed on `main` rather than going back to `release`. Neither fact was recorded anywhere, and several documents asserted the opposite. **The load-bearing correction is liveness.** `docs/tube-map-pipeline.html` said "None of this is live yet"; the roadmap's status table said "Live: Nothing"; the latency diagnosis framed its 120 s / 10 MB numbers as still current. All three now say what is true, and the before-numbers are labelled as the baseline they are rather than as the present. **The payload table was re-measured**, not copied forward. `perf/band-payload- sizes.mjs` after #66 took the segment outlines out of the header: the 8.0 kb region is 1.10 MB against 9.97 MB of SVG, where the roadmap still carried 1.25 MB and 8.0x, and the largest region is 9.3x rather than 9.0x. `docs/perf/increment-c.md` is new, and is C's counterpart to `increment-b.md`: what the response became, what deliberately did not move, the day pgb read it, and the three corrections that came back the other way while pgb's reader was being written from the spec (#67, #70, #66) — each a place where the format document and the encoder had already disagreed, and each caught by a person rather than by a test, which is the case for #25. **Two documents were wrong at the premise, and are amended rather than rewritten.** `docs/releasing.md` describes a discipline the deploy no longer follows; its banner now records why — one server, one port, no way to stand up an in-development instance beside the live one, so exercising a change against real data means pointing the live server at `main`. That is a reason to keep the branch, since a testing window needs a way home. The immediate hazard is named there too: `release` points at a commit nobody has run since August, so the documented rollback would replace working code with the 120-second version. ADR 0002 carries a dated amendment marking the two passages that no longer hold, including the consequence about a deploy validating a batch — which came true at 71 commits with no promotion message naming them. No code changed. The tag `increment-c` marks f9b05f6, the state pgb read. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FWNvEAujjzoxiM2RCLUV7W
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Documentation only — no code changed.
pgbread?format=bandsoff the live server on 2026-09-02 and drew from it, and the server stayed onmainafterwards instead of going back torelease. Neither fact was written down, and several documents asserted the opposite.What was wrong
tube-map-pipeline.htmlseqtubemap-roadmap.mdreleaseis 71 behindperf/seqtubemap-latency.md+.htmlperf/increment-b.mdperf/local-endpoint-harness.md+.htmlrelease— the branch the server followsRe-measured, not copied forward
perf/band-payload-sizes.mjswas re-run after #66 took the segment outlines out of the header. The roadmap's table was pre-#66 in two rows:New
docs/perf/increment-c.md, C's counterpart toincrement-b.md: what the response became, what deliberately did not move (#23 changed no bytes; #24 changed none on the SVG route; #66 moved 4–8 numbers per golden by ≤1.5e-16), the daypgbread it, and the three corrections that came back the other way whilepgb's reader was written from the spec rather than the encoder — #67, #70, #66. Each was a place the format document and the encoder had already disagreed, and each was caught by a person. That is the argument for #25.Amended at the premise, not rewritten
Two documents were built on "the server follows
release, so merging is not shipping", which is no longer true.releasing.mdgets a banner recording why the server is onmain, which is not drift: one server, one port, no second port available at the hosting facility, so the only way to exercise a change against real graph data is to point the live server atmainfor a window. That is an argument for keeping the branch — a testing window needs a way home. It also names the immediate hazard:releasepoints at a commit nobody has run since August, so the rollback procedure as documented would replace working code with the 120-second, request-blocking version. The procedure itself is untouched; whether to switch the server back, retire the branch for tags, or promote is a human decision.ADR 0002 gets a dated amendment marking the two passages that no longer hold, including its own consequence about a deploy validating a batch — which came true at 71 commits with no promotion message naming them.
Checked
tube-map-pipeline.htmlparse.The tag
increment-cmarksf9b05f6, the statepgbread from.🤖 Generated with Claude Code
https://claude.ai/code/session_01FWNvEAujjzoxiM2RCLUV7W