python-math #8: feat(math): replay harness H-B — Clojure Mode A driver (dev/replay.clj) + cross-language comparer shim - #2621
Draft
jucor wants to merge 1 commit into
Draft
Conversation
This was referenced Jul 18, 2026
jucor
marked this pull request as draft
July 18, 2026 02:29
This was referenced Jul 18, 2026
This was referenced Jul 18, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Adds the Phase H‑B replay harness pieces needed to produce Clojure “Mode A” step blobs and to reuse the existing Python step comparer to diff Clojure vs Python outputs without production-code changes. This supports the broader cross-language parity effort by creating a deterministic(ish) recording surface (prep-main / math_main blob shape) and a bridge for automated comparison.
Changes:
- Introduce a Clojure replay driver (
math/dev/replay.clj) and:replaydeps alias to generate per-stepprep-mainblobs (and optional full-state EDN) from schedule JSON + votes CSV. - Add a smoke script (
math/dev/replay_smoke.sh) to run the Clojure driver on the vw dataset and optionally run a cross-language compare against a Python recording. - Add Delphi-side shim + tests (
delphi/tests/replay_harness/clj_crosslang.py,test_clj_crosslang.py) to wrap Clojure blobs into the py-store shape and project both sides onto theprep-mainkey whitelist before diffing.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| math/dev/replay.clj | Clojure Mode A replay driver that slices votes per schedule, runs conv-update, and writes prep-main blobs + provenance. |
| math/dev/replay_smoke.sh | Developer smoke test to validate blob shape and optionally run cross-language compare. |
| math/deps.edn | Adds a :replay alias to run the driver without loading other dev namespaces. |
| delphi/tests/replay_harness/clj_crosslang.py | Test-only shim to treat Clojure step-*.blob.json as a py-store recording and to project blobs onto the prep-main whitelist for meaningful numeric diffs. |
| delphi/tests/replay_harness/test_clj_crosslang.py | Unit tests covering shim behavior and ensuring projected keys land on the numeric-compare path. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This was referenced Jul 25, 2026
Draft
Draft
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
…r (dev/replay.clj) + cross-language comparer shim commit-id:2b7f93f6
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.
commit-id:2b7f93f6
Stack: