feat: name leftover-map unexplained leftover on leftover pairs (v2.12.26) - #535
Conversation
….26) Persist unexplained leftover U = R − R̂ after two-axis Gabriel reconstruction so a leftover residual is not read as the leftover the map does not reconstruct. R̂ stays internal and is not persisted. Rank-0 maps store U = 0; fallback pairs omit U. After make seed, closest and farthest leftover pairs sit above the member list with U next to leftover-map distance d; click opens that post.
|
Merge contract for exact head
Requested reviewers stay empty: the only org collaborator is the author, so review cannot be requested from this account. |
|
Warning Review limit reachedNext included review available in 47 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (27)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
|
Cycle write probe 2026-08-24T07:35+09:00: leftover-map unexplained leftover (#535) stays independent. Next unique leftover honesty slice is unexplained share of centered leftover (s = U_c² / R̃², ADR 0183) on a fresh branch from main@ef6f5a5 — not mixed into this PR. Devin’s grand-mean-in-U note is addressed there by centering, not by patching this head. Author will not self-approve. Do not squash-merge without independent APPROVE + exact-head required success. Issues #79 and #87 stay open. |
…el render Login screen built returnUrl via raw string concatenation instead of returnUrlFromLocation()/rememberOidcReturnUrl(), leaving both imports unused (TS6192) and rendering an AdminPanel with accessToken: undefined (TS2322) on a branch guarded by !auth.isAuthenticated.
…xplained-v21226-local # Conflicts: # AGENTS.md # ARCHITECTURE.md # CHANGELOG.md # CLAUDE.md # backend/app/report_ingestion.py # backend/tests/test_api.py # docker/postgres-init/migrate.sh # docs/adr/0048-persist-lsirm-leftover-pairs.md # docs/adr/0049-leftover-pair-report-ui.md # frontend/package.json # frontend/src/App.test.tsx # frontend/src/App.tsx # frontend/src/api.ts # frontend/src/i18n.test.ts # frontend/src/i18n.ts # lineageweave/leftover_pairs.py # pyproject.toml # scripts/seed_demo_data.py # tests/test_leftover_pairs.py # tests/test_migration_replay.py # tests/test_period_report.py # tests/test_schema.py # uv.lock
# Conflicts: # AGENTS.md # ARCHITECTURE.md # CHANGELOG.md # CLAUDE.md # backend/tests/test_api.py # docs/adr/0003-fast-mlsirm-report-integration.md # frontend/package.json # lineageweave/leftover_pairs.py # pyproject.toml # tests/test_leftover_pairs.py # tests/test_period_report.py # tests/test_schema.py # uv.lock
1 participant
Buyer next action
After
make seed, leftover post–criterion pairs sit above the member list with unexplained leftover U next to leftover-map distance d. Click the pair to open that post. The U badge is omitted when the pair has no complete-case leftover map (fallback rows do not invent unexplained leftover). Never invent leftover score or theta.A leftover residual R is not the same fact as the leftover the two leftover-map axes do not reconstruct. Unexplained leftover names U = R − R̂ independently of leftover residual and of leftover-map distance. Two-axis reconstruction R̂ = ξ_{1:2} · ζ_{1:2} is computed internally and is not persisted.
Scope (this PR only)
leftover_map_unexplainedon leftover pair rowsleftover_map_unexplainedwhen complete-case ξ, ζ exist (U = R − R̂; unused axes pad with 0; hidden SVD axes after the second are dropped)leftover_map_reconstructionIndependent of leftover stacks already in review
Do not mix into:
Merge contract
2141a02b9d1d0e14ecfec940936cd24a0980400cAPPROVErequired (Copilot or human who is not the author)