Skip to content

python-math #51: fix+docs: rename stragglers (#2680 review) + quirks/queue freshness audit - #2683

Draft
jucor wants to merge 1 commit into
spr/edge/9547faa9from
spr/edge/98853895
Draft

python-math #51: fix+docs: rename stragglers (#2680 review) + quirks/queue freshness audit#2683
jucor wants to merge 1 commit into
spr/edge/9547faa9from
spr/edge/98853895

Conversation

@jucor

@jucor jucor commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Rename stragglers

The four stale 'delphi' values for math_env — the column that keys each math result row; the server only reads rows matching its own setting — that the #2680 review caught are now 'python': the compose comment, MATH_POLLER_DESIGN.md §4, CUTOVER_RUNBOOK.md step 2 (which contradicted step 1 mid-runbook), and the poller/__init__ docstring.

Doc freshness audit (Julien request)

  • Improvement-queue item 9 marked (a) DONE (PR python-math #48: feat(math): vectorize legacy kmeans hot paths (item 9a) #2679: ~70x speedup, bit-identical, verdict serial-OK) with (b) left optional; item 10 marked DONE (PR python-math #44: refactor(math): PR 14b/14c — readable two-phase stats + blob-injection pins #2673); item 12 added (persist warm-start state across restores — restart-induced K flips).
  • The queue header now points at the improvements/* park bookmarks (the jj bookmarks holding the deleted improved-mode branches for possible later re-landing).
  • The row for quirk Q11 — the legacy kmeans distance-cancellation quirk whose exact-0.0 ties decide cluster merges — gains the vectorization addendum: ties preserved bit-exactly; the dgemv/einsum kernels were rejected.
  • Quirks-row "legacy mode" phrasings are left as history — the preamble declares the mode collapse globally.

commit-id:98853895


Stack:


⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This was referenced Jul 27, 2026
This was referenced Jul 27, 2026
@jucor
jucor force-pushed the spr/edge/98853895 branch 3 times, most recently from edfadb5 to a6e40b0 Compare July 27, 2026 22:59
@jucor
jucor force-pushed the spr/edge/98853895 branch from a6e40b0 to 9ccf53e Compare July 27, 2026 23:07
@jucor
jucor force-pushed the spr/edge/98853895 branch from 9ccf53e to accfcf5 Compare July 28, 2026 00:11
@jucor jucor changed the title fix+docs: rename stragglers (#2680 review) + quirks/queue freshness audit python-math #51: fix+docs: rename stragglers (#2680 review) + quirks/queue freshness audit Jul 28, 2026
…queue freshness audit

## Rename stragglers

The four stale 'delphi' values for `math_env` — the column that keys each math result row; the server only reads rows matching its own setting — that the #2680 review caught are now 'python': the compose comment, `MATH_POLLER_DESIGN.md` §4, `CUTOVER_RUNBOOK.md` step 2 (which contradicted step 1 mid-runbook), and the `poller/__init__` docstring.

## Doc freshness audit (Julien request)

- Improvement-queue item 9 marked (a) DONE (PR #2679: ~70x speedup, bit-identical, verdict serial-OK) with (b) left optional; item 10 marked DONE (PR #2673); item 12 added (persist warm-start state across restores — restart-induced K flips).
- The queue header now points at the `improvements/*` park bookmarks (the jj bookmarks holding the deleted improved-mode branches for possible later re-landing).
- The row for quirk Q11 — the legacy kmeans distance-cancellation quirk whose exact-0.0 ties decide cluster merges — gains the vectorization addendum: ties preserved bit-exactly; the dgemv/einsum kernels were rejected.
- Quirks-row "legacy mode" phrasings are left as history — the preamble declares the mode collapse globally.

commit-id:98853895
@jucor
jucor force-pushed the spr/edge/98853895 branch from accfcf5 to 7ce20dd Compare July 28, 2026 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant