python-math #34: fix(delphi): Copilot-review triage batch — 16 findings across the stack - #2663
Draft
jucor wants to merge 1 commit into
Draft
python-math #34: fix(delphi): Copilot-review triage batch — 16 findings across the stack#2663jucor wants to merge 1 commit into
jucor wants to merge 1 commit into
Conversation
This was referenced Jul 27, 2026
Draft
Draft
Draft
Draft
Draft
Draft
Draft
This was referenced Jul 27, 2026
Draft
python-math #43: docs(delphi): s7 goal state + journal — mode collapse executed, battery 20/20
#2672
Draft
Draft
Draft
This was referenced Jul 28, 2026
Draft
jucor
changed the title
3 tasks
…gs across the stack
## What
Serial application of the triaged APPLY verdicts from the 18 Copilot reviews across the stack — 25 threads in total: 16 applied here, 2 rejected as replicated quirks citing the ledger (`CLOJURE_QUIRKS.md`), 3 declined, 1 deferred.
## Highlights
- Improved-mode early returns no longer leak stale `group_clusterings`/`group_k_smoother` across engine-mode switches (flipping a conversation between clojure-legacy and improved mode) — TDD with the failure observed RED first.
- The certify cache manifest now hashes comments CSVs (with the required one-time Clojure re-record of the 3 moderation battery entries, done).
- `lru_cache` on tree hashes.
- Battery-entry dataset-mismatch guard.
- Slug-glob sanitization.
- The quirk-Q11 clamp now propagates NaN (Q11: the Clojure k-means distance formula's cancellation floor — the clamp must floor only negative cancellation residue to 0.0, never mask NaN).
- Quirk-Q16 comment projection is always 2-wide (Q16: rank-1 principal components collapse all projections to [0,0]).
- Shard-bench file-descriptor/kill cleanup.
- Docstring + test hardening.
## Re-certified
Certification battery 20/20 MATCH x2 on this tree (`battery_s6_pass{1,2}.log`); the 647-test combined gate green.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
commit-id:efb914d7
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
1 participant
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.
What
Serial application of the triaged APPLY verdicts from the 18 Copilot reviews across the stack — 25 threads in total: 16 applied here, 2 rejected as replicated quirks citing the ledger (
CLOJURE_QUIRKS.md), 3 declined, 1 deferred.Highlights
group_clusterings/group_k_smootheracross engine-mode switches (flipping a conversation between clojure-legacy and improved mode) — TDD with the failure observed RED first.lru_cacheon tree hashes.Re-certified
Certification battery 20/20 MATCH x2 on this tree (
battery_s6_pass{1,2}.log); the 647-test combined gate green.Co-Authored-By: Claude Fable 5 noreply@anthropic.com
commit-id:efb914d7
Stack: