record(ORACLE-LLAMACPP-OWED-CPU): the boundary, the decision, and three counts that did not re-derive (#857) - #1075
Merged
Merged
Conversation
…heir word boundary back, and the instrument stays in this document (#1003) Both CPU-side items under this spec's `## Owed` are closed. Neither needed a host, which is why they are separable from the eleven that are blocked on one. The leading `\b` is restored on `FAV`'s two ratio branches. Measured on a clean detached worktree at `0f8580e269`, stage 1 goes from 1278 candidates in 146 files to 1260 in 145 and stage 2 from 1042 to 1024. The diff of the two candidate lists is 18 removals and zero insertions, and all 18 are adjudicated in the spec by shape and site. Every one is architecture-name or quantization text glued to a preceding word character, `C2x`, `sm8x` and `Q8_0 x Q8_0`, so no verdict leaves the enumeration. The question §"The two stages were one idea spelled twice" left open is answered, because it now has two readings to compare. Of the 38 candidate lines that match a ratio and are invisible to the retired pipe, the boundary drops 34 and keeps 4, and the four it keeps are exactly the fractional-leading-zero measurements `0.058x`, `0.086×` and `0.029×`. Restoring the boundary removes the glued text and keeps every real ratio, including every ratio below 1.0. The value rule stays deleted, because the axis and not the value decides polarity. Re-narrowing a gating expression owes a mutation pass, so all 43 were re-run against the block as committed rather than a copy of it: 43 red, 0 defects, plus two dict-reformat controls and the unmutated run green. Each row reports whether the edit applied, whether it still compiled, its exit code, and its exception class, because a mutation that never applied and one that fails to build both read as a passing test. The second item is a decision and it is against all three options. Moving the sweep to `tools/` is refused, though not for the reason the question assumed: `check-pr-size.py:510` attaches the mutation-evidence contract to `governance_checker` alone and `tools/*.py` classes as `product` at `:448`, so that contract would not attach. What refuses the move is that CI would enforce the self-test, and the self-test proves liveness and not shape coverage, which is the open defect. Scheduling that green raises its authority without changing its content. A per-shape control on every alternative is refused because a control is a string its author writes. A per-alternative census was measured and refused too: the historical dead branch reaches 5312 lines and its dead `×\b` half still reaches 470 against the repaired branch's 2121, so a census would have printed 470 rather than 1 and given a reader no threshold to act on. Also files #1058, which is not ours and is not fixed here. `main` fails `check-commit-trailers` on `e34d71379e70`, the #1054 squash, whose body carried no trailer block. History is not rewritable and `main` is squash-only, so the commit stands. `GATE-SQUASH-TRAILERS` owns it, because #870's CI `--filled` pull-request-body guard is the enforcement that would have refused the body. No number is re-taken, no row changes lifecycle state, and the residual under §"A live token is not a covered shape" stays open and stays stated. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [claude-code]
…railer gates can run The trailer and commit-style gates SKIP on a branch that is behind origin/main and still print nothing about the tree, which is #841 and #653. Merging is what makes them report. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [claude-code]
…rs this pass just wrote (#1003) Merging `origin/main` at `b493f4981` added 67 lines to `src/vt/cuda/cuda_quant_dot.cu`, so the `Q8_0×Q8_0` comment adjudicated at `:1067` is at `:1068` on the merged tree, and this pass's own new prose entered the sweep as a nineteenth removal. Re-run after the merge the drop is 19 removals and zero insertions, with no verdict among them, so the adjudication holds and only the line numbers moved. Recorded rather than silently corrected, because the table is a reading at `0f8580e269` and the point of naming the commit is that the reading drifts afterwards. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [claude-code]
…e trailer gates execute The trailer and commit-style gates SKIP on a branch behind origin/main and report nothing about the tree. Merged by the fresh reviewer to make them run. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [claude-code]
…es, and the 34/16 split carried no scope (#1003) Three findings from the fresh review of the word-boundary pass, all the same shape: a small count in a document whose own thesis is that a count without a re-derivable enumeration is untrustworthy. Each is repaired by re-deriving at `0f8580e269` on a clean detached worktree, not by editing the prose to agree with the review. The section now carries the derivation as an `sh` fence beside its prose, so a reader who runs the recipe reaches the same partition instead of taking three numbers on trust. Extracted from the committed spec and run verbatim, it prints `stop 35 leave 18 survive 17`, `blind 38 dropped 34`, and the four kept lines. The four ratios the boundary keeps are four LINES and three spellings. `0.058x` is on `.agents/benchmark-record.md:19021`, `.agents/specs/cpu-decode-barrier-and-attn-dispatch.md:33` and `.agents/specs/muse-glimmer.md:918`, and `.agents/specs/gguf-compute-in-quant-gemm.md:528` carries `0.086×` and `0.029×` on one line. The composition is 3+1 and the spec read it as 2+1+1. The spellings and the total of four were right. The stop-matching split had no scope on it. Over every baseline candidate, 35 lines stop matching a ratio, 18 leave the sweep and 17 survive. The 34 and 16 the spec gave are correct only inside the 38 lines invisible to the retired pipe. The line the two readings differ by is `.agents/coordination.md:1759`, which the retired pipe already saw on the `2x` of `SM100_MMA_F8F6F4_{SS,2x1SM_SS}`. Two antecedent slips are repaired. The deferral clause named two obligations and then said the repair carries all three, so it now names the third, an adjudication of every line the narrowing drops. The stale-anchor paragraph said two anchors moved when one did, and the nineteenth removal beside it is a new line rather than a moved anchor. Two things the review surfaced are recorded rather than left to be re-found. The adjudication table files each line under one shape, so its `sm8x, sm_12x` row under-counts `sm_12x` by four: `.agents/backend-matrix.md:172`, `.agents/parity-ledger.md:768`, `STATE-LEGACY-000001.md:27492` and `:27493` carry a CUTLASS generation too and are filed above. All five are glued text under either filing, so the adjudication does not move. And the extraction recipe's `[block] = [x for x in b if ...]` raises `ValueError` if a second python fence in this file ever carries both filter strings. It fails loud rather than selecting the wrong block, which is why the filter stays, and the fence this change adds is `sh` and cannot collide. The refusal to move the instrument to `tools/` gains the review's partial dissent and its revisit condition. The review agrees with the refusal, ranks §"Nothing lands dead" as the decisive reason and the `row/*` friction second, and calls "raises its authority" the weakest, because that one is about how a reader weighs a green. It also notes that a scheduled self-test WOULD catch a recurrence of the sixth hole. If #1003 gains a second consumer, the decisive argument evaporates and the move is to be taken again. No number is re-taken, no row changes lifecycle state, and the python sweep block is byte-unchanged by this commit, so the 43-mutation evidence stands. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [claude-code]
… so the trailer gates execute `origin/main` advanced to `100026481` while this repair was in flight, which made the branch behind it and turned `commit-trailers` and `commit-style` into SKIPs that reported nothing about this tree. Merging brings the two incoming commits in and lets both gates run. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [claude-code]
…n, and the same foreign-merge reason moved them (#1003) `## Gates` recorded 26 record gates, 45 mutation suites and 77 results with zero SKIP. Merging `origin/main` `10002648199cfbbaf1e423f7c80cacb2f4b56366` brings in `check-cuda-op-arch-gate` and `test_check_cuda_op_arch_gate` (#960, `d607fec4c`), so the same head now reports 27, 46 and 79. Measured on the merged head at loadavg 7.51 falling to 2.42: 79 ok, 0 FAIL, 0 SKIP, `All gates green`, with the trailer block executing rather than skipping because `origin/main` is an ancestor. The bullet already said this pair is a count a foreign merge changes. It has now changed twice, so the bullet names both moves and tells a reader to re-derive the pair per run rather than quote it. Neither number says anything about this branch. FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:claude-opus-5 [claude-code]
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.
FOLLOWING_AGENTS_PROTOCOL
Closes the two GPU-free
## Oweditems from the llama.cpp repin (#857,283c7e492) and files one issue. Records only: the diff is.agents/specs/oracle-llamacpp-repin-stock.mdand one appended.agents/issue-index.mdrow.The leading word boundary is restored
Stage 2 was rebuilt to compile from the same
FAVdict as stage 1, which fixed areal disagreement, but the retired pipe carried a leading
\bthatFAVdidnot. Restoring it, measured on a clean detached worktree at
0f8580e269with theinstrument extracted from the spec's own fence:
\brestored18 removals, zero insertions. All 18 adjudicated by shape and site: 14
C2x/C3x, 2sm8x/sm_12x, 2Q8_0, every one glued text with no verdictamong them.
The safety claim is the one that mattered, since trading a false positive for a
false negative is the unsafe direction. Verified structurally rather than by
sampling: each of the 18 carried exactly one stage-1 alternative, the loose
ratio, and zero other comparison tokens, with zero tight-ratio matches.
No verdict was lost.
It also closes a question the spec had left open. Of the 38 candidates that match
a ratio and are invisible to the retired pipe, the boundary drops 34 and keeps
4, and the four it keeps are exactly the fractional-leading-zero spellings. So
the boundary removes glued text and keeps every real ratio, including every one
below 1.0.
The instrument stays in this document, with the condition that would reverse it
The open question was whether to keep growing per-shape controls inside a
markdown fence or move the sweep to
tools/with a suite undertests/tools/.A caution I had raised turned out to be wrong, and was re-derived rather than
inherited.
check-pr-size.py:510attaches the mutation-evidence contract toclass
governance_checker, defined at:162asscripts/check-*.py|sh;tools/*.pyandtests/tools/test_*.pyboth fall through to:448asproduct. The contract would not have attached.What refuses the move is what CI would enforce instead.
tests/CMakeLists.txt:12auto-discovers
tests/tools/test_*.py, so the suite would run every build, andwhat it would schedule green is a self-test that proves liveness and not shape
coverage. Scheduling that green raises its authority without changing its
content, beside a paragraph saying the green is no evidence about coverage.
Secondarily,
check-role-discipline.py:63-71makestools/a feature prefix, soevery later edit needs a reviewed
row/*PR where.agents/is integration, andthe sweep has no consumer after #1003.
The decisive argument is the policy one: nothing lands dead. After #1003 the
instrument is reachable only from its own test, which proves the class works and
never that anything reaches it.
The condition that reverses this is recorded, because a refusal without one
reads as permanent: if #1003 ever gains a second consumer, the decisive argument
evaporates and the move to
tools/should be taken again. Also recorded is theweakest of the three reasons, and that a scheduled self-test genuinely would
catch a recurrence of the sixth hole.
Per-shape controls on every alternative are refused separately: a control is a
string its author writes, so seventeen invented shapes would read as coverage
while establishing nothing.
A third option killed itself under measurement
A per-alternative census was proposed, on the theory that an alternative with
near-empty reach is the signature of the
×defect. It was measured against thevery defect it was meant to catch: the historical dead branch reaches 5312
lines and its dead
×\bhalf still reaches 470, against the repairedbranch's 2121. It would have printed 470, not something near 1, giving no
threshold a reader could act on.
Rejected, with its numbers recorded in place of the feature. Measuring a proposed
instrument against the bug it is meant to catch is cheaper than shipping it and
discovering the same thing later.
Counts that did not survive re-derivation
Three small counts in this document were wrong, in a document whose own thesis is
that a count without a re-derivable enumeration is untrustworthy. They were
repaired by re-deriving, and the section now carries a second fence, extracted
verbatim and run, that prints every corrected number:
3+1rather than2+1+1, since one line carries two of them.34/18/16 reading is true only when scoped to the 38 lines invisible to the
retired pipe, and that scope is now stated, with the differing line named.
naming two, and one anchor moved rather than two.
The same class caught the row again while it was being repaired:
origin/mainlandedcheck-cuda-op-arch-gatemid-flight, moving the recordedgate blocks from 26/45/77 to 27/46/79. Both moves are now named, with the reader
told to re-derive the pair per run rather than trust either.
Also recorded: the adjudication table assigns one shape per line, so
sm_12xappears on five lines with four filed under
C2x/C3x; all five are glued texteither way, so the adjudication does not move. And the recipe's single-fence
filter raises
ValueErrorif a second matching python fence is ever added tothis spec, which fails loud rather than selecting the wrong block, and which
keeping the instrument in a document inherits.
Filed
#1058 —
e34d71379e70carriesnone of the required trailer lines, verified by re-running the checker rather
than by taking the report. Owned by
GATE-SQUASH-TRAILERS, since #870's CI--filledPR-body guard is the missing enforcement. The commit is not repaired,because history is not rewritable here.
Gates
scripts/agent-preflight.sh: 79 ok, 0 FAIL, 0 SKIP, exit 0, gated againstorigin/main 100026481, named in both range headings, with the trailer blockexecuted rather than skipped.
The instrument's own mutation pass was rebuilt independently in review: 43 red,
3 green, 0 defects, every red row reporting
applied=Truewith a non-zero linedelta,
compiles=True,rc=1. The two dict-regeneration controls apply cleanlyand stay green, which is what shows the rewrite mechanism is not what turns the
other rows red.
Owed
The remaining
## Oweditems on this row all need hardware and are unchanged:the twelve re-takes under #1003, the keep-f16 default's decision, and a chosen
revision for Laguna's Poolside figure before it can be re-taken at all.
Following-Agents-Protocol: true
AI-Assisted: true
Assisted-by: AGENT:claude-opus-5 [Claude Code]