Skip to content

feat(reports): leftover-map singular values on leftover-map graphic-display axes (v2.81.0) - #865

Open
seonghobae wants to merge 1 commit into
feat/leftover-map-compare-axis-singular-v2800from
feat/leftover-map-plot-axis-singular-v2810
Open

feat(reports): leftover-map singular values on leftover-map graphic-display axes (v2.81.0)#865
seonghobae wants to merge 1 commit into
feat/leftover-map-compare-axis-singular-v2800from
feat/leftover-map-plot-axis-singular-v2810

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Caption leftover-map graphic-display axes with persisted leftover-map singular values σ_k (ADR 0324 / v2.81.0).
  • After make seed, leftover-map graphic-display axes name persisted leftover-map singular values when finite together with persisted leftover-map axis share; click a leftover pair opens that post.
  • Leftover-map graphic-display axes stay leftover-map axis {k} σ {value} ({share}%) so they stay distinct from leftover-axis leftover axis {k} σ {value} {share}% (ADR 0322), from comparison leftover-axis leftover map comparison leftover axis {k} σ {value} {share}% (ADR 0323), and from comparison graphic leftover map comparison graphic leftover-map axis {k} σ {value} ({share}%) (ADR 0321).
  • A missing, non-finite, or negative singular value omits that σ caption and keeps leftover-map axis {k} ({share}%). Rank-0 unused axes still name σ 0.00. Do not invent σ_k from leftover-map axis share.
  • UI-only; no SQL; no new columns. Never invent a leftover score. Never invent a theta. Do not persist leftover-map inner product, cosine, or length. This increment does not change leftover-map singular values on leftover-axis report badges or on leftover-axis report badges on the grouping comparison strip.
  • Stacked onto leftover-map singular values on comparison leftover-axis badges (feat/leftover-map-compare-axis-singular-v2800 / feat(reports): leftover-map singular values on comparison leftover-axis badges (v2.80.0) #863 @ 52b3597). Leave this PR open for independent review. Do not squash-merge onto the unprotected leftover base. Do not self-approve. After independent review, retarget onto leftover-coordinates (feat/leftover-map-coordinates-v2240 / feat(reports): persist leftover-map coordinates (v2.24.0) #782) rather than squash-merging here. Issues Milestone 2: port verified direct-PostgreSQL analysis into the protected product architecture #79 and Milestone 2.1: add normalized runtime-analysis schema bridge after v0.77 #87 stay open. Do not merge revert: undo #782 squash until independent APPROVE #808 without independent APPROVE.

Test plan

  • leftoverMapPlotAxisSingular.test.ts leftover-map graphic-display axis badges read 1.84/0.82, name rank-0 0, omit NaN/Infinity/null/negative independently of share, do not invent σ from leftover-map axis share, stay distinct from leftover-axis / comparison leftover-axis / comparison graphic copy
  • LeftoverMapPlot.test.tsx leftover-map graphic names leftover-map axis 1 σ 1.84 (82%) / leftover-map axis 2 σ 0.86 (18%); rank-0 leftover-map axis 1 σ 0.00 (0%); missing share leftover-map axis 1 σ 1.84; omitted σ keeps leftover-map axis 1 (82%)
  • LeftoverPairList.test.tsx leftover-map graphic names leftover-map axis 1 σ 1.84 (82%) / leftover-map axis 2 σ 0.86 (18%)
  • App.test.tsx period leftover-map graphic names leftover-map axis 1 σ 1.84 (82%) / leftover-map axis 2 σ 0.86 (18%); grouping comparison keeps leftover map comparison graphic leftover-map axis 1 σ 1.84 (82%) and leftover map comparison leftover axis 1 σ 1.84 82% and omits leftover-map axis 1 σ 1.84 (82%)
  • i18n.test.ts required keys for leftover-map graphic-display σ + share in ko/zh/ja/vi
  • frontend oxlint on changed files: 0 warnings, 0 errors
  • frontend tsc -b passed
  • frontend vitest: 933 tests passed
  • CI Full test suite and Frontend lint, test, build on this exact head
  • After make seed, leftover-map graphic-display axes name leftover-map singular values; click opens that post.

Evidence boundary

Authenticated PostgreSQL API and k6 runtime acceptance are not claimed. No real records or credentials. Synthetic fixtures only.

Independent approval is required. No self-approval, admin bypass, or force push.

Jeon, M., Jin, I. H., Schweinberger, M., & Baugh, S. (2021). Mapping unobserved item–respondent interactions: A latent space item response model with interaction map. Psychometrika, 86(2), 378–403.

Gabriel, K. R. (1971). The biplot graphic display of matrices with application to principal component analysis. Biometrika, 58(3), 453–467.


Devin Review

…isplay axes (v2.81.0)

Caption leftover-map graphic-display axes with persisted leftover-map singular
values σ_k (ADR 0324). Graphic copy stays leftover-map axis {k} σ {value}
({share}%) so it stays distinct from leftover-axis leftover axis {k} σ {value}
{share}%, from comparison leftover-axis leftover map comparison leftover axis
{k} σ {value} {share}%, and from comparison graphic leftover map comparison
graphic leftover-map axis {k} σ {value} ({share}%). Rank-0 unused axes still
name σ 0.00. Share and singular value omit independently. Do not invent σ_k
from leftover-map axis share. Never invent a leftover score. Never invent a
theta.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 11:20 KST. Exact head a3613ac (v2.81.0 / ADR 0324). Caption leftover-map graphic-display axes with persisted leftover-map singular values σ_k. Distinct copy leftover-map axis {k} σ {value} ({share}%). Rank-0 unused axes still name σ 0.00. Share and singular omit independently. Do not invent σ_k from leftover-map axis share. Independent APPROVE required. Do not self-approve. Leave stacked leftover PRs open. Do not squash-merge onto the unprotected leftover base. After independent review, retarget onto leftover-coordinates (#782, MERGED). Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE. Synthetic fixtures only.

@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5608fb6b-4425-478f-b98e-2bf877423ffa

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 11:16 KST. Writes work. Exact head a3613ac (v2.81.0 / ADR 0324). Frontend lint, test, build success; Full test suite still in progress (do not wait on Checks). Devin COMMENTED is not independent APPROVE. Copilot/CodeRabbit skipped on non-default base. Leave #865 open. Do not squash-merge. Do not self-approve. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE. Independent APPROVE required on this exact head. Next buyer increment after this stacked σ_k surface: leftover-map singular values on leftover-map pair-list leftover-axis / remaining share-only leftover-map axis captions, or the next already-persisted leftover-map quantity that still has no buyer caption. Do not persist leftover-map inner product, cosine, or length. Synthetic fixtures only.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 11:16 KST. Writes work as seonghobae. Exact head a3613ac (v2.81.0 / ADR 0324). Frontend lint/test/build SUCCESS; Full test suite SUCCESS. Devin COMMENTED is not independent APPROVE. Copilot/CodeRabbit skipped on non-default base. Leave #865 open. Do not squash-merge. Do not self-approve. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE. Independent APPROVE required on this exact head.

Next buyer increment stacked locally this cycle: leftover-map singular values on leftover-axis report badges independently of leftover-map axis share (ADR 0325 / v2.82.0) on feat/leftover-map-axis-singular-only-v2820. Period leftover-axis badges still coupled σ and share (NaN% when share is missing). Distinct copy leftover axis {k} σ {value} when share is omitted. Rank-0 unused axes still name σ 0.00. Share and singular omit independently. Do not invent σ_k from leftover-map axis share. Do not invent leftover-map axis share from σ_k. Do not persist leftover-map inner product, cosine, or length. Synthetic fixtures only.

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