python-math #39: feat(math): mode collapse C3 — powerit warm-start PCA + legacy kmeans as the only solvers (item 8 parked) - #2668
Draft
jucor wants to merge 1 commit into
Draft
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
This was referenced Jul 27, 2026
Draft
Draft
Draft
Draft
This was referenced Jul 28, 2026
Draft
jucor
changed the title
3 tasks
…A + legacy kmeans as the only solvers (item 8 parked) ## What `GOAL_CUTOVER_READY.md` Phase 2, chunk C3 of the mode collapse (removing the engine's improved-mode branches so the Clojure-faithful behavior is the only code path). The engine's solver paths are now unconditionally the Clojure-faithful ones: - `_compute_pca`: always warm-starts power iteration from the previous tick's components (Clojure's `:start-vectors`, `conversation.clj:385`), with `require_powerit=True`. - `_compute_clusters`: the base level always runs `legacy_kmeans` with lineage warm start (PR-C of this stack); the group level always runs the per-k legacy loop with the group-K smoother (PR-D). The sklearn cold-recompute arms (`kmeans_sklearn` at the base level + `best_k` group selection) are deleted; dead `GROUP_ITERS`/`base_weights`/`legacy_mode` cleaned up. - `POLISMATH_PCA_IMPL` (the env var selecting the PCA implementation) is left in `pca.py` but is now ENGINE-INERT: with `require_powerit` always True, a sklearn selection is always overridden back to powerit (the existing warn+fallback). Full removal ships with queue item 8 post-cutover — deleting it now would cascade through 9 test files for zero behavior change (scope ruling, journal session 7). ## Tests Improved-mode pins deleted with their branches: improved-tick2-cold, cross-mode cold-tick equality (PCA warm start), improved-never-calls-legacy-kmeans + cross-mode vw invariance (lineage; replaced by a legacy determinism pin), and 3 smoother-inert tests. 51 tests green across the affected files. The deleted sklearn arms are queue item 8 in `POST_CUTOVER_IMPROVEMENTS.md` (park commit at the end of the collapse). commit-id:30038853
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
GOAL_CUTOVER_READY.mdPhase 2, chunk C3 of the mode collapse (removing the engine's improved-mode branches so the Clojure-faithful behavior is the only code path). The engine's solver paths are now unconditionally the Clojure-faithful ones:_compute_pca: always warm-starts power iteration from the previous tick's components (Clojure's:start-vectors,conversation.clj:385), withrequire_powerit=True._compute_clusters: the base level always runslegacy_kmeanswith lineage warm start (PR-C of this stack); the group level always runs the per-k legacy loop with the group-K smoother (PR-D). The sklearn cold-recompute arms (kmeans_sklearnat the base level +best_kgroup selection) are deleted; deadGROUP_ITERS/base_weights/legacy_modecleaned up.POLISMATH_PCA_IMPL(the env var selecting the PCA implementation) is left inpca.pybut is now ENGINE-INERT: withrequire_poweritalways True, a sklearn selection is always overridden back to powerit (the existing warn+fallback). Full removal ships with queue item 8 post-cutover — deleting it now would cascade through 9 test files for zero behavior change (scope ruling, journal session 7).Tests
Improved-mode pins deleted with their branches: improved-tick2-cold, cross-mode cold-tick equality (PCA warm start), improved-never-calls-legacy-kmeans + cross-mode vw invariance (lineage; replaced by a legacy determinism pin), and 3 smoother-inert tests. 51 tests green across the affected files. The deleted sklearn arms are queue item 8 in
POST_CUTOVER_IMPROVEMENTS.md(park commit at the end of the collapse).commit-id:30038853
Stack: