python-math #42: feat(delphi): mode collapse C7 — harness identifier purge + flag machinery deletion - #2671
Draft
jucor wants to merge 1 commit into
Draft
python-math #42: feat(delphi): mode collapse C7 — harness identifier purge + flag machinery deletion#2671jucor wants to merge 1 commit into
jucor wants to merge 1 commit into
Conversation
This was referenced Jul 27, 2026
python-math #43: docs(delphi): s7 goal state + journal — mode collapse executed, battery 20/20
#2672
Draft
Draft
Draft
Draft
Draft
Draft
Draft
Draft
Draft
This was referenced Jul 27, 2026
Draft
Draft
Draft
Draft
This was referenced Jul 28, 2026
Draft
jucor
changed the title
3 tasks
…purge + flag machinery deletion ## What Final chunk of the `GOAL_CUTOVER_READY.md` Phase 2 mode collapse — the removal of the engine-mode switch between clojure-legacy (the setting that reproduces the old Clojure engine exactly) and improved behavior. The goal doc's DONE-gate grep (`ENGINE_MODE|engine_mode|resolve_engine_mode` over `delphi/polismath/`) now returns ZERO hits. ## Changes - `certify.py` (the certification-battery harness): `BatteryEntry` / `derive_schedule_id` / `parse_battery_entry` drop the mode. Schedule ids keep their historical `-clojure-legacy` suffix via the frozen `_LEGACY_SUFFIX` literal, and fingerprints bake the same literal into the digest — so recording directories AND every historical key in `divergences.json` (the ledger of observed Python-vs-Clojure divergences) stay valid. The manifest drops its mode key, riding the same Python-side re-record that the engine change already forced. `run_py_driver` no longer sets any env; `compare_recording_pair` drops its dead mode parameter; battery reports and ledger observations drop the mode field (historical ledger values keep theirs on disk). - `poller_equiv.py` (the live Python-vs-Clojure poller equivalence harness): env plumbing deleted (`build_py_env` / `PyPollerRunner` / `EquivConfig`). `store.py` provenance drops the env record. - `polismath/utils/engine_mode.py` DELETED; `scripts/certify_battery.json` entries drop the key. - Test sweep: engine-mode imports, fixtures, and setenv calls removed across 20 test files; `test_engine_mode.py` and the two env-guard conftest fixtures deleted; 9 default-mode tests re-pinned to the now-only legacy semantics (arrival-order tid export, inert bans, bucket votes-base, the 1x1 real-math edge case, base-cluster-id (bid) group members, greedy-floor-aware D2c); the D9/D10 comparisons on the vw dataset's cold-start variant now XPASS (parity improved). ## Testing Full suite: 1155 passed / 22 skipped / 44 xfailed / 2 xpassed. commit-id:7f42df81
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
Final chunk of the
GOAL_CUTOVER_READY.mdPhase 2 mode collapse — the removal of the engine-mode switch between clojure-legacy (the setting that reproduces the old Clojure engine exactly) and improved behavior. The goal doc's DONE-gate grep (ENGINE_MODE|engine_mode|resolve_engine_modeoverdelphi/polismath/) now returns ZERO hits.Changes
certify.py(the certification-battery harness):BatteryEntry/derive_schedule_id/parse_battery_entrydrop the mode. Schedule ids keep their historical-clojure-legacysuffix via the frozen_LEGACY_SUFFIXliteral, and fingerprints bake the same literal into the digest — so recording directories AND every historical key indivergences.json(the ledger of observed Python-vs-Clojure divergences) stay valid. The manifest drops its mode key, riding the same Python-side re-record that the engine change already forced.run_py_driverno longer sets any env;compare_recording_pairdrops its dead mode parameter; battery reports and ledger observations drop the mode field (historical ledger values keep theirs on disk).poller_equiv.py(the live Python-vs-Clojure poller equivalence harness): env plumbing deleted (build_py_env/PyPollerRunner/EquivConfig).store.pyprovenance drops the env record.polismath/utils/engine_mode.pyDELETED;scripts/certify_battery.jsonentries drop the key.test_engine_mode.pyand the two env-guard conftest fixtures deleted; 9 default-mode tests re-pinned to the now-only legacy semantics (arrival-order tid export, inert bans, bucket votes-base, the 1x1 real-math edge case, base-cluster-id (bid) group members, greedy-floor-aware D2c); the D9/D10 comparisons on the vw dataset's cold-start variant now XPASS (parity improved).Testing
Full suite: 1155 passed / 22 skipped / 44 xfailed / 2 xpassed.
commit-id:7f42df81
Stack: