Skip to content

feat(reports): leftover-map unexplained leftover share on grouping comparison graphic (v2.69.0) - #852

Draft
seonghobae wants to merge 4 commits into
feat/leftover-map-compare-plot-explained-share-v2680from
feat/leftover-map-compare-plot-unexplained-share-v2690
Draft

feat(reports): leftover-map unexplained leftover share on grouping comparison graphic (v2.69.0)#852
seonghobae wants to merge 4 commits into
feat/leftover-map-compare-plot-explained-share-v2680from
feat/leftover-map-compare-plot-unexplained-share-v2690

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Caption persisted leftover-map unexplained leftover share s on the grouping-comparison leftover-map graphic when formatLeftoverMapUnexplainedShare returns a usable badge (ADR 0312 / v2.69.0). It omits independently from explained share, reconstruction, distance, and coverage. No SQL/new columns; never derive s from U/R or neighbouring statistics.

Live authority

  • parent/base: #851@72df615a8fb1350217039c6a62007243a6322ccc
  • current product head: 80b9254aea2177f2bb506acf0370688df424a036
  • PR: open / Draft

Parent convergence

#852 inherited the ADR-governance regression as ed6502e2b214b8a1d6c3463d8febb32a049028e5. GitHub generated exact current-parent merge 27c2330729414736a6f8fb5af9d8ffbf9ae0c35c, verified with parents #851@72df615a... and #852@ed6502e2...; the merge tree preserves ADR 0312/v2.69.0 while carrying ADR 0304 Proposed. That tree was recorded child-first as 80b9254a...; branch movement was non-force only.

Historical 808 frontend tests/oxlint/py_compile evidence predates this exact head and is not transferred. Exact-head required/frontend/browser/a11y, PostgreSQL/k6 evidence, and independent approval remain outstanding. Direct descendants must be reconstructed before predecessor evidence transfers.

Product invariants

  • Missing/non-finite s omits only its graphic caption; values above 1 remain visible and are not clamped.
  • Explained share/reconstruction/distance/coverage remain independent.
  • This increment does not add a distinct cross-share graphic caption.

Keep Draft until exact-head evidence and independent approval exist. No self-approval, gate bypass, force push, or destructive rebase.

…mparison graphic (v2.69.0)

Caption persisted leftover-map unexplained leftover share s on the grouping
comparison leftover-map graphic from already-named leftover_map_unexplained_share
(ADR 0312). Comparison copy uses leftover map comparison graphic unexplained
leftover share {label} so it stays distinct from hyphen leftover-map unexplained
leftover share {label} and from strip Leftover map comparison unexplained leftover
share. A missing or non-finite s omits that caption and keeps leftover-map
explained leftover share, leftover-map reconstruction, leftover-map distance,
and comparison graphic coverage notes. Rank-0 origin cells still name U²/R² 0.00
when that share is finite. Do not caption leftover-map cross share this increment.
Do not invent s from U and R or leftover-map distance. Never invent a leftover
score. Never invent a theta. No new columns.

Copy link
Copy Markdown
Contributor Author

Queue retry: leftover-map unexplained leftover share on grouping comparison graphic (v2.69.0 / ADR 0312) @ 2c34896. Independent APPROVE required before squash-merge. Do not self-approve. Do not squash-merge onto the unprotected leftover base. After independent review, retarget onto leftover-coordinates (#782, already on main).

Verification: vitest 808 passed, oxlint 0, py_compile ok. Next stacked increment is leftover-map cross share on that comparison graphic. Issues #79 and #87 stay open. #96 is already closed. Do not merge #808 without independent APPROVE.

@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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

Queue retry: leftover-map unexplained leftover share on grouping comparison graphic (v2.69.0 / ADR 0312). Independent APPROVE required before squash-merge. Do not self-approve. Next stacked increment is leftover-map cross share on that comparison graphic. Issues #79 and #87 stay open. #96 is already closed.

@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 2 potential issues.

Devin Review

Comment on lines +353 to +358
aria-label={tf(
variant === "comparison"
? LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_UNEXPLAINED_SHARE
: LEFTOVER_MAP_PLOT_SEGMENT_UNEXPLAINED_SHARE,
{ label: segment.unexplainedShareLabel },
)}

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: Variant switch preserves omission behavior

LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_UNEXPLAINED_SHARE changes only the accessible name. Missing shares still omit independently because rendering remains gated by the existing formatter.

Devin Review

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

Comment thread frontend/src/i18n.ts
"leftover map comparison graphic explained leftover share {label}":
"잔여 지도 비교 그림 설명 잔여 점유율 {label}",
"Leftover map comparison explained leftover share": "잔여 지도 비교 설명 잔여 점유율",
"leftover map comparison graphic unexplained leftover share {label}": "잔여 지도 비교 그림 미설명 잔여 점유율 {label}",

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: Translations follow runtime lookup order

Every non-English locale defines the new key. English safely uses the key itself before tf substitutes the formatted persisted value.

Devin Review

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

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31T06:40 KST: writes work. Independent APPROVE still required; not self-approving. Local duplicate of this increment was 8d3085c on a shallow clone of #851 and was not force-pushed. Next buyer gap after this head: leftover-map cross share on the grouping comparison leftover-map graphic (ADR 0313 / v2.70.0).

Copy link
Copy Markdown
Contributor Author

Cycle retry (2026-08-31T07:08 KST): write-scope probe. Independent APPROVE still required; not self-approving.

Copy link
Copy Markdown
Contributor Author

Queue retry 2026-08-31T07:21 KST: leftover-map unexplained leftover share on grouping comparison graphic remains stacked at v2.69.0. Independent APPROVE required on exact head 2c34896. Do not self-approve. Do not squash-merge onto the unprotected leftover base. Next buyer gap is leftover-map cross share on the grouping comparison graphic (distinct comparison-graphic name; strip already has ADR 0296).

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 ~07:08 KST: writes retry. Head 2c34896 still has no independent APPROVE (Devin Review COMMENT only). Leave #852 open for independent review. Do not squash-merge. Do not self-approve. Next buyer increment is leftover-map cross share on grouping comparison leftover-map graphic (ADR 0313 / v2.70.0) stacked onto this branch. Issues #79 and #87 stay open. Do not merge #808.

Copy link
Copy Markdown
Contributor Author

Stacked buyer increment opened: leftover-map cross share on grouping comparison leftover-map graphic (v2.70.0 / ADR 0313) is #853 @ c4f79bc on feat/leftover-map-compare-plot-cross-share-v2700. Leave both PRs open for independent review. Do not squash-merge. Do not self-approve.

@seonghobae
seonghobae marked this pull request as draft September 4, 2026 08:06
Adopt the conflict-free GitHub test-merge tree for exact parent #851 89fe034 while preserving existing #852 product head 2c34896 as first-parent ancestry. No force push and no product delta is dropped.
seonghobae added a commit that referenced this pull request Sep 6, 2026
Adopt the conflict-free GitHub test-merge tree for exact parent #852 78d809a while preserving existing #853 product head c4f79bc as first-parent ancestry. No force push and no product delta is dropped.
Adopt the GitHub-verified conflict-free merge tree for exact parent #85172df615 and child ed6502e. The tree preserves ADR 0312/v2.69.0 and inherits ADR 0304 Proposed plus its regression. No force push and no valid delta is dropped.
seonghobae added a commit that referenced this pull request Sep 6, 2026
Adopt the GitHub-verified conflict-free merge tree for exact parent #85280b9254 and child 3f53449. The tree preserves ADR 0313/v2.70.0 and inherits ADR 0304 Proposed plus its regression. No force push and no valid delta is dropped.
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