Skip to content

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

Draft
seonghobae wants to merge 11 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 11 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 (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.

Current exact authority

Parent convergence and child RED → repair

#851's explained-share Vietnamese semantics and generated-lock repair are inherited normally rather than copied. The convergence commit also preserves #852's v2.69 product delta and repairs the child-owned Vietnamese accessible-name defect for LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_UNEXPLAINED_SHARE: production copy keeps explicit share semantics (tỷ phần phần dư chưa giải thích ...) instead of reducing the label to an unexplained residual phrase.

The synthetic pre-merge validation of #1000 exposed a real merge-result lock-parity RED (uv.lock 2.68.0 vs pyproject.toml 2.69.0). The promoted two-parent product commit does not inherit that stale generated artifact: current pyproject.toml, runtime lineageweave.__version__, and editable uv.lock are all 2.69.0. Do not transfer the failed synthetic merge receipt or predecessor GREEN as acceptance for de9e7b3....

Descendant convergence

The prior body was stale. Direct child #853 has already adopted this exact repaired parent non-force and is now at ef02b1d2fab4f9f7a8db797a29acda93e156847d; #854 is based on that current #853 head at 746bd6a102fa6ec6eb181779369e98cdad7fa34d, with later serialized descendants converged onward through #858/#859. Do not create duplicate restacks for already-current ancestry. Parent movement remains a repair event, not a close reason, and no child may copy parent-owned repairs independently. Each descendant keeps its own exact-head RED/gate state; #859 currently owns the comparison-distance consumer-test RED.

Product invariants

  • Missing/non-finite persisted s omits only its graphic caption; finite zero and 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.
  • Canonical KO/EN/JA/ZH/VI/ES/DE/FR translation authority remains the governed Customer Master path; this PR consumes/represents screen copy and does not create a competing store.

Keep Draft until this exact head has fresh repository/security validation, required browser/a11y/performance evidence, and qualifying independent approval. No self-approval, bypass, force push, destructive rebase, generated-lock hand edit, ancestor skip, or predecessor-evidence transfer.

…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

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: Advanced

Run ID: a7444f0b-e656-497e-a947-aeab8acac30b

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

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.
Adopt exact parent #8510ec2828 without rewriting #852. Preserve the #852 product tree and inherit the project/uv lock version-parity RED verbatim. No force push or product delta is dropped.
seonghobae added a commit that referenced this pull request Sep 10, 2026

@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.

현재 exact head의 v2.69 제품 경로에서 Vietnamese LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_UNEXPLAINED_SHARE 번역이 phần dư chưa giải thích ...로 되어 있어 unexplained residual만 말하고 share 의미를 보존하지 않습니다. Production constant는 leftover map comparison graphic unexplained leftover share {label}이고 formatter 값도 U²/R², 즉 비율/몫입니다. 기존 leftoverMapUnexplainedShare.test.ts는 영문 key identity와 수치 formatter만 검사해 이 locale 의미 손실을 잡지 못합니다. #851의 explained-share 번역 repair를 먼저 상속한 뒤, 이 PR 고유 s 경로에도 production-key Vietnamese regression을 추가하고 tỷ phần phần dư chưa giải thích ...처럼 share 의미가 명시되는 최소 번역 repair를 수행해야 합니다. 다른 locale·measurement semantics·owner boundary는 변경하지 않습니다.

Adopt #851 repaired explained-share/lock delta, preserve #852 unexplained-share product delta, and repair its Vietnamese production accessible-name share semantics after an exercised RED.
@seonghobae
seonghobae marked this pull request as ready for review September 10, 2026 21:52
@seonghobae
seonghobae marked this pull request as draft September 10, 2026 21:58
github-actions Bot pushed a commit that referenced this pull request Sep 10, 2026
…tics

Adopt exact repaired #852, preserve #853 cross-share product delta, regenerate v2.70 lock with repository-authoritative uv, and repair the exercised Vietnamese accessible-name share-semantic RED.
seonghobae added a commit that referenced this pull request Sep 10, 2026
…tics

Adopt exact repaired #852 de9e7b3 as the second parent while preserving #853 product delta from 4eae847. Promote the workflow-free tree that exercised the Vietnamese cross-share accessible-name RED, repaired explicit share semantics, regenerated uv.lock with uv 0.11.28 to v2.70.0, and passed focused frontend/build/Storybook plus project lock parity validation.
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