Skip to content

feat(reports): leftover-map singular values on comparison leftover-axis badges (v2.80.0) - #863

Open
seonghobae wants to merge 1 commit into
feat/leftover-map-axis-singular-v2790from
feat/leftover-map-compare-axis-singular-v2800
Open

feat(reports): leftover-map singular values on comparison leftover-axis badges (v2.80.0)#863
seonghobae wants to merge 1 commit into
feat/leftover-map-axis-singular-v2790from
feat/leftover-map-compare-axis-singular-v2800

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • leftoverMapPlotAxisSingular.test.ts comparison leftover-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 / leftover-map graphic / comparison graphic copy
  • leftoverMapAxisBadge.test.ts period leftover-axis copy stays distinct from comparison leftover-axis copy
  • App.test.tsx grouping comparison names leftover map comparison leftover axis 1 σ 1.84 82% / leftover map comparison leftover axis 2 σ 0.86 18%; period leftover-axis leftover axis 1 σ 1.84 82% remains and stays out of the comparison strip
  • LeftoverMapPlot.test.tsx comparison graphic still omits leftover map comparison leftover axis 1 σ 1.84 82%; omitted σ uses leftover_singular_value: Number.NaN
  • i18n.test.ts required keys for comparison leftover-axis σ + share in ko/zh/ja/vi
  • frontend oxlint on changed files: 0 warnings, 0 errors
  • frontend tsc -b passed
  • frontend vitest: 576 tests passed (App.test.tsx, LeftoverMapPlot.test.tsx, LeftoverPairList.test.tsx, i18n.test.ts, leftover-map helpers)
  • CI Full test suite and Frontend lint, test, build on this exact head
  • After make seed, grouping comparison leftover-axis report badges 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

…is badges (v2.80.0)

Caption leftover-axis report badges on the grouping comparison strip with
persisted leftover-map singular values σ_k (ADR 0323). Comparison copy stays
leftover map comparison leftover axis {k} σ {value} {share}% so it stays
distinct from period leftover-axis leftover axis {k} σ {value} {share}%
(ADR 0322). Rank-0 unused axes still name σ 0.00. Share and singular value
omit independently. Do not invent σ_k from leftover-map axis share. Fail-closes
leftover-map comparison graphic TypeScript on omitted leftover-map singular
values. Never invent a leftover score. Never invent a theta.
@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: 1ff663c1-8c14-4ca8-aa13-ff1002065034

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.

Copy link
Copy Markdown
Contributor Author

Exact head 52b3597e5c6b825f988a9c8a7c68ae1653535982 stacked on #862 @ f24b664. Copilot/Devin review is not independent APPROVE. Leave this PR open. Do not squash-merge. Do not self-approve. Issues #79 and #87 stay open.

@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

Writes work this cycle. Exact head 52b3597e5c6b825f988a9c8a7c68ae1653535982 still has no independent APPROVE (Copilot/Devin COMMENTED is not independent APPROVE). Leave #863 open. Do not squash-merge. Do not self-approve. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE. Next buyer increment is leftover-map singular values on leftover-map graphic-display axes (ADR 0324 / v2.81.0): period leftover-map graphic axes still name share-only leftover-map axis {k} ({share}%) (ADR 0322). Caption persisted σ_k there with distinct leftover-map axis {k} σ {value} ({share}%). Stack onto this head. Do not persist leftover-map inner product, cosine, or length.

Copy link
Copy Markdown
Contributor Author

Cycle write probe 2026-08-31T02:16Z: requesting independent APPROVE on exact head only. Do not self-approve. Strix fail-closed remains in force for Vulnerabilities [1-9].

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