Skip to content

feat(reports): leftover-map axis share on leftover-map comparison leftover-axis ticks independently of leftover-map singular values (v2.90.0) - #874

Open
seonghobae wants to merge 1 commit into
feat/leftover-map-plot-tick-share-badge-v2890from
feat/leftover-map-compare-axis-tick-share-badge-v2900
Open

feat(reports): leftover-map axis share on leftover-map comparison leftover-axis ticks independently of leftover-map singular values (v2.90.0)#874
seonghobae wants to merge 1 commit into
feat/leftover-map-plot-tick-share-badge-v2890from
feat/leftover-map-compare-axis-tick-share-badge-v2900

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Caption leftover-map comparison leftover-axis ticks with persisted leftover-map axis share through leftoverMapCompareAxisTickBadge independently of leftover-map singular values (ADR 0333 / v2.90.0).
  • After make seed, leftover-map comparison leftover-axis ticks name persisted leftover-map axis share when leftoverMapCompareAxisTickBadge returns a usable leftover-axis tick leftover-map axis share caption even when leftover-map singular values are omitted; click a leftover pair opens that post.
  • Leftover-map comparison leftover-axis ticks stay leftover map comparison leftover axis {k} tick {value} {share}% when leftover-map axis share is finite and σ_k omits, and leftover map comparison leftover axis {k} tick {value} σ {singular} {share}% when both are finite so they stay distinct from leftover-map graphic leftover-map axis ticks leftover-map axis {k} tick {value} σ {singular} {share}% (ADR 0332), from leftover-map comparison graphic leftover-map axis ticks leftover map comparison graphic leftover-map axis {k} tick {value} σ {singular} {share}% (ADR 0331), from leftover-axis leftover axis {k} {share}% (ADR 0325), from comparison leftover-axis leftover map comparison leftover axis {k} {share}% (ADR 0323), from leftover-axis ticks leftover axis {k} tick {value} σ {singular} (ADR 0330), and from leftover-map comparison leftover-axis ticks leftover map comparison leftover axis {k} tick {value} σ {singular} (ADR 0329).
  • A missing or non-finite leftover-map axis share omits that leftover-map axis share tick caption and keeps leftover map comparison leftover axis {k} tick {value} or leftover map comparison leftover axis {k} tick {value} σ {singular}. Rank-0 unused axes still name leftover-map axis share 0%. Share and singular value omit independently. Do not invent leftover-map axis share from σ_k. 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-axis ticks, leftover-map graphic leftover-map axis ticks, or leftover-map comparison graphic leftover-map axis ticks.
  • Stacked onto leftover-map graphic leftover-map axis tick leftover-map axis share as leftoverMapPlotTickAxisBadge (feat/leftover-map-plot-tick-share-badge-v2890 / feat(reports): leftover-map axis share on leftover-map graphic leftover-map axis ticks independently of leftover-map singular values (v2.89.0) #873 @ 923bcfe). 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 leftoverMapCompareAxisTickBadge names 82%/18% with σ 1.84/0.86, share-only when σ omits, rank-0 share 0%, omits NaN/null share independently of σ, does not invent share from σ or σ from share, leftover-axis ticks stay σ-only
  • App.test.tsx grouping comparison leftover-axis tick leftover map comparison leftover axis 1 tick +0.50 σ 1.84 82%; leftover-axis ticks stay leftover axis 1 tick +0.50 σ 1.84
  • i18n.test.ts ko/zh/ja/vi share and singular-share leftover-axis tick keys
  • oxlint + tsc -b + vitest leftover-map + App (693 passed)

Devin Review

…tover-axis ticks independently of leftover-map singular values (v2.90.0)

Caption leftover-map comparison leftover-axis ticks with persisted leftover-map axis share through leftoverMapCompareAxisTickBadge independently of leftover-map singular values (ADR 0333). Share and σ omit independently. Rank-0 unused axes still name leftover-map axis share 0%. Leftover-axis ticks stay σ-only. Do not invent leftover-map axis share from σ_k. Do not invent σ_k from leftover-map axis share. Never invent a leftover score or a theta. No new columns.
@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: a83b4310-9749-404c-b6a1-a3b195a620f9

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

Leave open for independent APPROVE. Do not self-approve. Frontend lint and leftover-map + App tests are green (693 passed, oxlint 0, tsc -b). After independent review, retarget onto leftover-coordinates rather than squash-merging onto the leftover base. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE.

@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 14:40 KST: leftover-map comparison leftover-axis tick leftover-map axis share (ADR 0333 / v2.90.0) is complete at ba58c21. Frontend lint SUCCESS, Full test suite SUCCESS, Devin COMMENT only. Leave open for independent APPROVE. Do not self-approve. Do not squash-merge onto the leftover base. After independent review, retarget onto leftover-coordinates rather than merging here. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE. Next stacked increment: leftover-axis ticks leftover-map axis share independently of leftover-map singular values as leftoverMapAxisTickBadge (ADR 0334 / v2.91.0).

Copy link
Copy Markdown
Contributor Author

cycle-write-probe 2026-08-31T06:16Z: independent review only; no self-approve. Next buyer gap after leftover-map axis share (v2.90.0).

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 15:24 KST: writes work (ghu_ seonghobae, admin/push). This PR #874 remains MERGEABLE at ba58c21 (v2.90.0 / ADR 0333). Frontend lint SUCCESS, Full test suite SUCCESS. Devin COMMENT is not independent APPROVE. Do not self-approve. Do not squash-merge onto the leftover base. Next increment is leftover-axis ticks leftover-map axis share independently of leftover-map singular values as leftoverMapAxisTickBadge (ADR 0334 / v2.91.0) on feat/leftover-map-axis-tick-share-badge-v2910 stacked onto this head. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE.

Copy link
Copy Markdown
Contributor Author

Leave open for independent APPROVE. Do not self-approve. Devin found no issues. After independent review, retarget onto leftover-coordinates rather than squash-merging onto the leftover base. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE.

Copy link
Copy Markdown
Contributor Author

Cycle note (2026-08-31 15:24 KST): GitHub writes work (ghu_, seonghobae, admin/push). Devin COMMENT on exact head ba58c21 is not independent APPROVE. Frontend lint SUCCESS; Full test suite SUCCESS. Do not self-approve. Do not squash-merge this leftover-map stack onto unprotected leftover base. Next increment: leftover-axis ticks leftover-map axis share independently of leftover-map singular values as leftoverMapAxisTickBadge (ADR 0334 / v2.91.0) on feat/leftover-map-axis-tick-share-badge-v2910 stacked onto this PR. This increment does not change leftover-map comparison leftover-axis ticks, leftover-map graphic leftover-map axis ticks, or leftover-map comparison graphic leftover-map axis ticks. #96 CLOSED. #79 and #87 stay open. #808 stays draft.

Copy link
Copy Markdown
Contributor Author

Cycle retry 2026-08-31T15:43 KST: exact head ba58c21 (v2.90.0 leftover-map comparison leftover-axis tick leftover-map axis share). Independent APPROVE still required on this exact head. Do not self-approve. Do not squash-merge onto the unprotected leftover base. Issues #79 and #87 stay open.

Copy link
Copy Markdown
Contributor Author

Leave #874 open for independent APPROVE. Devin Review COMMENTED is not APPROVE. Do not squash-merge. Do not self-approve. After independent review, retarget leftover-map stack onto leftover-coordinates (feat/leftover-map-coordinates-v2240 / #782) rather than squash-merging here. Next stacked increment: leftover-axis ticks leftover-map axis share independently of leftover-map singular values as leftoverMapAxisTickBadge (ADR 0334 / v2.91.0) on feat/leftover-map-axis-tick-share-badge-v2910. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE.

Copy link
Copy Markdown
Contributor Author

Leave #874 open for independent APPROVE. Stacked next increment is leftover-axis ticks leftover-map axis share independently of leftover-map singular values as leftoverMapAxisTickBadge (ADR 0334 / v2.91.0) on feat/leftover-map-axis-tick-share-badge-v2910 / #875 @ 8fa6042. Do not squash-merge. Do not self-approve. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 15:53 KST: #875 leftover-axis ticks leftover-map axis share independently of leftover-map singular values (ADR 0334 / v2.91.0) is stacked onto this exact head ba58c21. This PR #874 remains MERGEABLE (v2.90.0 / ADR 0333). Devin COMMENT is not independent APPROVE. Leave open for independent APPROVE. Do not self-approve. Do not squash-merge onto the leftover base. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE.

Copy link
Copy Markdown
Contributor Author

Leave #874 open for independent APPROVE. Do not squash-merge. Do not self-approve. Devin COMMENTED is not APPROVE. #875 leftover-axis tick leftover-map axis share (v2.91.0 / ADR 0334) remains stacked on this head. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE.

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