Repository navigation
docs(analog): mos_psp103 and mos_table gate notes say what was measured (Closes #12518) - #12519
Merged
Merged
Conversation
…e measured gate (Closes #12518) Prose only. The comment headers of the two accepted files and the "Not done" section of the mos_table note still said the mutation gate had not been run, while the notes record a measured gate run with exit 0. - mutation-accepted-mos_psp103.txt: the "gate NOT run" line is replaced by the measured gate result copied from its note (359 mutants, 306 killed, 30 survived, 18 hung, 5 unviable; 48 not killed, 48 accepted, 0 accepted lines now killed) and a pointer to the note; the census line is labelled as the census. - mutation-accepted-mos_table.txt: "Predicted ... not measured" becomes the measured gate result (632, 583, 25, 20, 4; 45 not killed, all accepted), next to the census (632, 493, 115, 20, 4). - mutation-mos_table-2026-10-25.md, "Not done": the gate bullet is turned into "ran and passed"; the other bullets were checked, see below. Verified before keeping or dropping each "Not done" bullet: - Seal: .trinity/seals/analog_AnalogMosTable.json was re-saved in the merged lane PR #12401 after the batch 3 tests; on master it records 37 of 37 tests, 0 vacuous, and its spec_hash equals the sha256 of specs/analog/mos_table.t27 (shasum). The "no seal was saved" bullet is corrected. - Coverage and docs check: the lane did not run them (kept as stated); the CI coverage job of #12401 reported a failure, cause not examined, and the bullet says so without blaming this spec. - Group K and line 835: the "Gate result (measured)" section lists none of their lines among the not-killed mutants, so the "gate remains" wording is dropped. - The 20 hung: the gate reports 20 hung and all 45 not-killed mutants are accepted; the bullet no longer says they were not re-run. No non-comment line of either accepted file changed (checked with git diff -U0 and a grep for non-comment +/- lines: empty); both files keep their line counts. ASCII only, no home path. No spec, seal, test or workflow touched. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ps a coverage claim it did not examine (Refs #12518) The "Not done" bullet quoted a commit title that is not in master history; it now names PR 12401 and its squash commit 96ce237 (the seal file log shows that commit). The bullet that blamed a CI coverage job of that PR for a failure rested on a skimmed log, so it now says only that coverage and the docs check were not run in this lane. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Contributor
Summary by t27-beesThis PR updates three analog mutation documentation files to replace predicted gate results with actual measured results from running the mutation gate. The headers now quote measured gate counts (killed/survived/hung/unviable, accepted/not-killed) and reference the corresponding gate result sections in the dated markdown files. Worth a look
Read: 3 files, +16 -13, the whole diff. Head |
Collaborator
Author
|
Merge basis (hand-merge under the rule of 2026-10-25 16:50, owner veto default from 2026-10-26 12:00 +07), head 412bfaa (master dc98d95 is already in it):
🤖 Addressed by 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.
Summary
mutation-accepted-mos_psp103.txt: the header no longer says the gate was not run; it quotes the measured gate result of the note (359 mutants, 306 killed, 30 survived, 18 hung, 5 unviable; 48 accepted, 0 stale).mutation-accepted-mos_table.txt: "predicted, not measured" is replaced by the measured gate result (632 mutants, 583 killed, 25 survived, 20 hung, 4 unviable; 45 accepted) next to the census.mutation-mos_table-2026-10-25.md, section "Not done": made to agree with its own section "Gate result (measured)"; the seal bullet was checked against the tree (seal re-saved in PR 12401, squash commit 96ce237, top-levelspec_hashequals the sha256 of the spec, 0 vacuous), and a coverage claim that rested on a skimmed log was dropped.Closes #12518
Refs #10486
🤖 Generated with Claude Code