docs(applied): record the 96cfea7 corpus-sweep discharge; add braid-mode smoke (#300) - #385
Merged
Conversation
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.
Closes #300.
The 100k-pair published-figure corpus (
catgraph-applied/tests/smc_nf_differential_sweep.rs+tests/content_equality_corpus.rs) is guarded day-to-day by a 5k default-mode smoke; the issue's phase-1 probe cited an undated run at96cfea7claiming all five--ignoredsweeps passed, with no durable record of it in the repo.Change
96cfea7claim (unverifiable — I never ran the tests at that commit), added a dated module-doc section tosmc_nf_differential_sweep.rsrecording a fresh, independently-verified pass of the same five sweeps at this branch's base (89241cc, 2026-08-31):published_divergence_figures_reproduce,published_braid_mode_figures_reproduce,published_interleave_mode_figures_reproduce(this file) pluspublished_corpus_is_closed_by_content,braid_mode_corpus_is_closed_by_content(content_equality_corpus.rs) — 5/5 pass in one combined--ignoredrun, 15.13s. The96cfea7claim is cited as prior context, not asserted as independently checked.braid_mode_smoke_prefix_of_the_published_corpus— a non-ignored 5k-pairMode::Braidprefix, pinningCounts { divergent: 77, in_fragment: 38, marked: 17 }(discovered by running the code, not estimated), mirroring the existingMode::Defaultsmoke test. Closes the issue's stated gap: the §4.6Countsdivergence-bucket figures for braid mode were previously reachable only via the--ignored100k tier.Review (2 rounds,
rust-v2:falsifying-reviewer, perturb-in-copy)89241cc→1d92c6a, record + smoke test): no blocking/important. One minor: the new smoke test overlaps with a pre-existing, already-CI-visible braid-mode test —canonical_display_corpus.rs'ssmoke_prefix_braid_corpus(predates this branch, same seed, non-ignored, inci.yml) — which tracks a different statistic (Gatereadback/churn) than this file'sCountsdivergence bucket. Not a defect (the new test'sCountscoverage is genuinely new), but the doc comment's framing needed to disclose the overlap rather than imply this is the first braid-mode CI check. Confirmed the record's own claims check out: re-running the cited command reproduced 5/5 passes (14.96s, consistent with the doc's 15.13s), and bothcontent_equality_corpus.rstest names it cites are real. Confirmed the new smoke test is non-vacuous (perturbingcolumn_pair_is_admissible'shas_braidguard reddens it,Counts{66,36,17}vs expected{77,38,17}) and not redundant with the default-mode smoke test (same perturbation leaves that one green —Mode::Defaultnever carries braid atoms). Applied in99acb53.1d92c6a→99acb53delta): no findings — comment-only diff verified, every clause of the disclosure checked against the code (smoke_prefix_braid_corpus's location, non-ignored status, seed, andGate's disjoint field set vsCounts) and against git history (smoke_prefix_braid_corpustraced to commit2268dc0, smc_nf: canonicalize η placement (ι) — the single remaining canonicality freedom on braid-free diagrams #187, well before this branch). Converged.Every perturbation round ran in a throwaway
cp -rcopy with its own--target-dir; the worktree stayed clean throughout.Gates (worktree,
--target-dir /tmp/cargo/catgraph-G5-T4)cargo nextest run -p catgraph-applied: 669 passed, 0 failed, 12 skipped (default features)cargo nextest run -p catgraph-applied --features internal-probes: 686 passed, 0 failed, 20 skipped — includes the newbraid_mode_smoke_prefix_of_the_published_corpuscargo clippy --all-targets -- -D warnings: clean on default,--no-default-features,--features internal-probesRUSTDOCFLAGS="-D warnings" cargo doc -p catgraph-applied --no-deps: cleancargo fmt --all --check: cleanscripts/check_measured_claims.py/scripts/check_audit_counts.py: not applicable — diff carries noMEASUREDmarkers and touches no audit doc🤖 Generated with Claude Code