Skip to content

feat(math): moderation + restart-seam replay coverage — mod_update parity, Q13-Q16#2656

Draft
jucor wants to merge 1 commit into
spr/edge/90ba0c34from
spr/edge/9467ac51
Draft

feat(math): moderation + restart-seam replay coverage — mod_update parity, Q13-Q16#2656
jucor wants to merge 1 commit into
spr/edge/90ba0c34from
spr/edge/9467ac51

Conversation

@jucor

@jucor jucor commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Battery coverage for the goal-doc moderation-heavy, meta-tids and
restart-seam edge cases, plus the quirks they surfaced (all TDD, ledgered
in CLOJURE_QUIRKS.md; design in MOD_RESTART_PORT_SPEC.md):

  • Conversation.mod_update: Clojure mod-update reducer parity
    (conversation.clj:846-884) — is_meta lands in BOTH mod sets,
    un-moderation disj, order-sensitive, watermark max floor.
  • Q15: legacy recompute() drops the mod watermark (conv-update's graph
    has no :last-mod-timestamp node).
  • Q16: rank-1 comps collapse ALL projections to [0,0] (utils/zip
    truncation on nil pc2) — replicated at both projection helpers; found
    by review-driven 1xN/Nx1 tiny-shape tests with Clojure-derived values.
  • group-votes tally source: raw-rating-mat (conversation.clj:601-608),
    both the compute path and the duplicate inline to_dict tally.
  • replay drivers: mod-event weaving (votes-then-moderation per batch,
    one blob per step) + load-or-init restart seam in BOTH drivers
    (blob round-trip via db/load-conv's longs-else-keywords key-fn,
    restructure-json-conv, full-history raw-rating-mat, mod-update);
    clj seam validated bit-identical pre-seam vs cached recordings.
  • prodclone extractor: additive is-meta/modified comment columns;
    certify passes --comments for moderation schedules.
  • battery: pc-revote-02 swap (Q13 carve-out), + pc-modheavy-01,
    pc-meta-01, vw-restart4, pc-midmix-restart3 (19 entries).

Q13 (split-loop knife-edge ties) and the two OPEN mod-entry residuals
(uniqify id-lineage; distinct-row-count fork) are documented in the
journal What's Next.

Co-Authored-By: Claude Fable 5 noreply@anthropic.com

commit-id:9467ac51


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 22, 2026
@jucor
jucor force-pushed the spr/edge/90ba0c34 branch from a628d9e to c94d602 Compare July 22, 2026 11:23
@jucor
jucor force-pushed the spr/edge/9467ac51 branch 2 times, most recently from c36adb0 to 5456492 Compare July 22, 2026 11:40
@jucor
jucor force-pushed the spr/edge/90ba0c34 branch from c94d602 to 069d052 Compare July 22, 2026 11:40
@jucor
jucor force-pushed the spr/edge/9467ac51 branch from 5456492 to adf1ea1 Compare July 22, 2026 13:27
@jucor
jucor force-pushed the spr/edge/90ba0c34 branch from 069d052 to 3803dfb Compare July 22, 2026 13:27
…rity, Q13-Q16

Battery coverage for the goal-doc moderation-heavy, meta-tids and
restart-seam edge cases, plus the quirks they surfaced (all TDD, ledgered
in CLOJURE_QUIRKS.md; design in MOD_RESTART_PORT_SPEC.md):

- Conversation.mod_update: Clojure mod-update reducer parity
  (conversation.clj:846-884) — is_meta lands in BOTH mod sets,
  un-moderation disj, order-sensitive, watermark max floor.
- Q15: legacy recompute() drops the mod watermark (conv-update's graph
  has no :last-mod-timestamp node).
- Q16: rank-1 comps collapse ALL projections to [0,0] (utils/zip
  truncation on nil pc2) — replicated at both projection helpers; found
  by review-driven 1xN/Nx1 tiny-shape tests with Clojure-derived values.
- group-votes tally source: raw-rating-mat (conversation.clj:601-608),
  both the compute path and the duplicate inline to_dict tally.
- replay drivers: mod-event weaving (votes-then-moderation per batch,
  one blob per step) + load-or-init restart seam in BOTH drivers
  (blob round-trip via db/load-conv's longs-else-keywords key-fn,
  restructure-json-conv, full-history raw-rating-mat, mod-update);
  clj seam validated bit-identical pre-seam vs cached recordings.
- prodclone extractor: additive is-meta/modified comment columns;
  certify passes --comments for moderation schedules.
- battery: pc-revote-02 swap (Q13 carve-out), + pc-modheavy-01,
  pc-meta-01, vw-restart4, pc-midmix-restart3 (19 entries).

Q13 (split-loop knife-edge ties) and the two OPEN mod-entry residuals
(uniqify id-lineage; distinct-row-count fork) are documented in the
journal What's Next.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

commit-id:9467ac51
@jucor
jucor force-pushed the spr/edge/9467ac51 branch from adf1ea1 to 1f327c5 Compare July 22, 2026 13:48
@jucor
jucor force-pushed the spr/edge/90ba0c34 branch from 3803dfb to 0f364eb Compare July 22, 2026 13:48
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