Skip to content

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

Draft
seonghobae wants to merge 1 commit into
feat/leftover-map-axis-tick-badge-v2870from
feat/leftover-map-compare-plot-tick-share-badge-v2880
Draft

feat(reports): leftover-map axis share on leftover-map comparison graphic leftover-map axis ticks independently of leftover-map singular values (v2.88.0)#872
seonghobae wants to merge 1 commit into
feat/leftover-map-axis-tick-badge-v2870from
feat/leftover-map-compare-plot-tick-share-badge-v2880

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Caption leftover-map comparison graphic leftover-map axis ticks with persisted leftover-map axis share through leftoverMapComparePlotTickAxisBadge independently of leftover-map singular values (ADR 0331 / v2.88.0).
  • After make seed, leftover-map comparison graphic leftover-map axis ticks name persisted leftover-map axis share when leftoverMapComparePlotTickAxisBadge returns a usable leftover-map 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 graphic leftover-map axis ticks stay leftover map comparison graphic leftover-map axis {k} tick {value} {share}% when leftover-map axis share is finite and σ_k omits, and leftover map comparison graphic leftover-map 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} (ADR 0327), from leftover-map graphic leftover-map axis leftover-map axis {k} σ {value} ({share}%) (ADR 0324), from leftover-axis leftover axis {k} {share}% (ADR 0325), from comparison leftover-axis leftover map comparison leftover axis {k} {share}% (ADR 0323), from comparison graphic leftover-map axis leftover map comparison graphic leftover-map axis {k} σ {value} ({share}%) (ADR 0326), from comparison leftover-axis ticks leftover map comparison leftover axis {k} tick {value} σ {singular} (ADR 0329), from leftover-axis ticks leftover axis {k} tick {value} σ {singular} (ADR 0330), and from comparison graphic leftover-map axis ticks leftover map comparison graphic leftover-map axis {k} tick {value} σ {singular} (ADR 0328).
  • A missing or non-finite leftover-map axis share omits that leftover-map axis share tick caption and keeps leftover map comparison graphic leftover-map axis {k} tick {value} or leftover map comparison graphic leftover-map 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-map graphic leftover-map axis ticks, leftover-map comparison leftover-axis ticks, or leftover-axis ticks.
  • Stacked onto leftover-axis tick leftover-map singular values as leftoverMapAxisTickBadge (feat/leftover-map-axis-tick-badge-v2870 / feat(reports): leftover-map singular values on leftover-axis ticks independently of leftover-map axis share (v2.87.0) #871 @ e9e4c2a). 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 leftoverMapComparePlotTickAxisBadge 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, leftoverMapPlotTickAxisBadge stays σ-only
  • LeftoverMapPlot.test.tsx comparison ticks with share present tick +0.50 σ 1.84 82% / tick −0.02 σ 0.86 18% / rank-0 tick 0.00 σ 0.00 0%; σ-omitted+share-present share-only tick +0.50 82%; NaN-share still σ-only; graphic ticks stay σ-only
  • App.test.tsx grouping comparison tick leftover map comparison graphic leftover-map axis 1 tick +0.50 σ 1.84 82%
  • i18n.test.ts ko/zh/ja/vi share and singular-share tick keys
  • oxlint + tsc -b + vitest leftover-map + App (659 passed)

Devin Review

…phic leftover-map axis ticks independently of leftover-map singular values (v2.88.0)

Caption leftover-map comparison graphic leftover-map axis ticks with persisted leftover-map axis share through leftoverMapComparePlotTickAxisBadge independently of leftover-map singular values (ADR 0331). Share and σ omit independently. Rank-0 unused axes still name leftover-map axis share 0%. Leftover-map graphic leftover-map axis ticks, leftover-map comparison leftover-axis ticks, and 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: dc78e323-8ac2-47be-a4da-5779b2a6ab4c

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

Independent review required on exact head 7bdc562. Do not self-approve. Do not squash-merge onto the unprotected leftover base. Stacked onto #871 @ e9e4c2a. Leave #869/#870/#871 open. 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

@seonghobae seonghobae left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Leave open for independent APPROVE. Do not self-approve. Frontend lint is green; Full test suite still running. After independent review, retarget onto leftover-coordinates rather than squash-merging onto the leftover base.

Copy link
Copy Markdown
Contributor Author

Cycle write probe 2026-08-31T05:25Z: retry GitHub writes before next leftover-map buyer gap. Independent APPROVE still required on exact head 7bdc562. Do not self-approve.

Copy link
Copy Markdown
Contributor Author

Cycle write probe 2026-08-31T05:25Z. Leave #872 open for independent APPROVE on exact head 7bdc562. Do not self-approve. Next buyer gap is leftover-map graphic leftover-map axis ticks with leftover-map axis share independently of leftover-map singular values.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31T05:24Z: writes retry on #872. Independent APPROVE still required on this leftover-map comparison graphic leftover-map axis tick leftover-map axis share increment (v2.88.0 / ADR 0331). Do not self-approve. Do not squash leftover stack onto unprotected leftover base. Next stacked increment is leftover-map graphic leftover-map axis ticks with leftover-map axis share independently of leftover-map singular values (leftoverMapPlotTickAxisBadge, v2.89.0 / ADR 0332) onto this head. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 14:40 KST: leftover-map comparison graphic leftover-map axis tick leftover-map axis share (ADR 0331 / v2.88.0) remains open. Independent APPROVE required. Do not self-approve. Do not squash-merge. Stack continues through #873 and #874. Issues #79 and #87 stay open.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 15:24 KST: leave open for independent APPROVE. Do not self-approve. Do not squash-merge. Next increment after #874 is leftover-axis tick leftover-map axis share (ADR 0334 / v2.91.0). Issues #79 and #87 stay open.

Copy link
Copy Markdown
Contributor Author

Cycle note (2026-08-31 15:24 KST): leftover-map stack continues. Next increment ADR 0334 / v2.91.0 leftover-axis tick leftover-map axis share. Do not self-approve. Do not squash-merge this stack onto unprotected leftover base.

@seonghobae
seonghobae marked this pull request as draft September 4, 2026 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant