Skip to content

feat(reports): leftover-map origin on leftover-map comparison graphic leftover-map axis ticks independently of leftover-map axis share and leftover-map singular values (v2.92.0) - #877

Draft
seonghobae wants to merge 1 commit into
feat/leftover-map-axis-tick-share-badge-v2910from
feat/leftover-map-compare-plot-tick-origin-badge-v2920
Draft

feat(reports): leftover-map origin on leftover-map comparison graphic leftover-map axis ticks independently of leftover-map axis share and leftover-map singular values (v2.92.0)#877
seonghobae wants to merge 1 commit into
feat/leftover-map-axis-tick-share-badge-v2910from
feat/leftover-map-compare-plot-tick-origin-badge-v2920

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Caption leftover-map comparison graphic leftover-map axis origin ticks through leftoverMapComparePlotTickAxisBadge independently of leftover-map axis share and leftover-map singular values (ADR 0335 / v2.92.0).
  • After make seed, leftover-map comparison graphic leftover-map axis origin ticks name leftover-map origin 0.00 when leftoverMapComparePlotTickAxisBadge returns a usable leftover-map origin tick caption even when leftover-map axis share or leftover-map singular values are omitted; click a leftover pair opens that post.
  • Leftover-map comparison graphic leftover-map axis origin ticks stay leftover map comparison graphic leftover-map axis {k} origin tick {value} when leftover-map axis share and σ_k omit, leftover map comparison graphic leftover-map axis {k} origin tick {value} {share}% when leftover-map axis share is finite and σ_k omits, leftover map comparison graphic leftover-map axis {k} origin tick {value} σ {singular} when σ_k is finite and leftover-map axis share omits, and leftover map comparison graphic leftover-map axis {k} origin tick {value} σ {singular} {share}% when both are finite so they stay distinct 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-map graphic leftover-map axis ticks leftover-map axis {k} tick {value} σ {singular} {share}% (ADR 0332), from leftover-map comparison leftover-axis ticks leftover map comparison leftover axis {k} tick {value} σ {singular} {share}% (ADR 0333), from leftover-axis ticks leftover axis {k} tick {value} σ {singular} {share}% (ADR 0334), and from leftover-map comparison graphic leftover-map axis leftover map comparison graphic leftover-map axis {k} σ {value} ({share}%) (ADR 0326).
  • Rank-0 unused axes still name leftover-map origin 0.00 and leftover-map axis share 0%. Share and singular value omit independently. Do not invent leftover-map origin from leftover-map axis share or σ_k. 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 axis share as leftoverMapAxisTickBadge (feat/leftover-map-axis-tick-share-badge-v2910 / feat(reports): leftover-map axis share on leftover-axis ticks independently of leftover-map singular values (v2.91.0) #875 @ d19f1fd). 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 leftover-map origin 0.00 with σ 1.84/0.86 and share 82%/18%, share-only when σ omits, σ-only when share omits, rank-0 origin share 0%, does not invent origin from share or σ, distinct from leftoverMapPlotTickAxisBadge / leftoverMapCompareAxisTickBadge / leftoverMapAxisTickBadge / leftoverMapComparePlotAxisBadge
  • LeftoverMapPlot.test.tsx comparison graphic leftover-map axis origin tick leftover map comparison graphic leftover-map axis 1 origin tick 0.00 σ 1.84 82%; leftover-map graphic leftover-map axis ticks stay leftover-map axis {k} tick {value}
  • i18n.test.ts ko/zh/ja/vi leftover-map comparison graphic leftover-map axis origin tick keys
  • oxlint + tsc -b + vitest leftover-map + App + i18n (737 passed)

Devin Review

… leftover-map axis ticks (v2.92.0)

Caption leftover-map comparison graphic leftover-map axis origin ticks through leftoverMapComparePlotTickAxisBadge independently of leftover-map axis share and leftover-map singular values (ADR 0335). Origin 0.00 stays leftover map comparison graphic leftover-map axis {k} origin tick {value} so buyers can tell the rank-0 unused-axis location from leftover-map coordinate ticks. Share and σ omit independently. Do not invent leftover-map origin from share or σ. UI-only; no SQL; no leftover-map inner product, cosine, or length.
@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: 3e1e467b-17bc-4204-8893-4a90d72c44d1

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 this PR open for independent APPROVE. Do not squash-merge onto the unprotected leftover base. Do not self-approve. Devin/Copilot COMMENT is not APPROVE.

Stacked onto leftover-axis tick leftover-map axis share (feat/leftover-map-axis-tick-share-badge-v2910 / #875 @ d19f1fd). After independent review, retarget leftover-map stack onto leftover-coordinates (feat/leftover-map-coordinates-v2240 / #782) rather than squash-merging here. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE.

Cycle 2026-08-31 16:22 KST: GitHub writes work as seonghobae. #96 already closed (duplicate of #91). #875 left open (Devin COMMENTED, not APPROVE). This increment is leftover-map origin on leftover-map comparison graphic leftover-map axis ticks (ADR 0335 / v2.92.0). oxlint + tsc -b + vitest leftover-map + App + i18n 737 passed. Never invent leftover/theta. Do not persist leftover-map inner product, cosine, or length.

@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 found 1 potential issue.

Devin Review

Comment on lines +152 to +155
export function leftoverMapPlotTickIsOrigin(tickLabel: string): boolean {
const originLabel = formatSignedLeftoverValue(0);
return originLabel !== null && tickLabel === originLabel;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 Info: Rounded coordinates preserve origin identity

uniqueCoordinateTicks inserts zero before persisted coordinates and deduplicates their formatted labels. A tiny nonzero coordinate cannot replace the true origin.

Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

This was referenced Aug 31, 2026
seonghobae added a commit that referenced this pull request Sep 3, 2026
…r-axis independently of leftover-map comparison graphic leftover-map origin (v2.114.0)

Caption leftover-map comparison leftover-axis leftover-map origin through leftoverMapCompareAxisOriginBadge (ADR 0357). Distinct accessible name leftover map comparison leftover axis leftover-map origin {origin}. Rank-0 unused axes still name leftover-map origin (0.00, 0.00). leftoverMapAxisOriginBadge stays unnamed. No SQL. Do not invent leftover-map origin from ζ, leftover-map axis share, or σ_k. Do not mix #877. Do not mix #890.
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