Skip to content

feat(reports): leftover-map unexplained leftover on grouping comparison graphic (v2.71.0) - #854

Open
seonghobae wants to merge 11 commits into
feat/leftover-map-compare-plot-cross-share-v2700from
feat/leftover-map-compare-plot-unexplained-leftover-v2710
Open

feat(reports): leftover-map unexplained leftover on grouping comparison graphic (v2.71.0)#854
seonghobae wants to merge 11 commits into
feat/leftover-map-compare-plot-cross-share-v2700from
feat/leftover-map-compare-plot-unexplained-leftover-v2710

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Caption persisted leftover-map unexplained leftover U on the grouping-comparison graphic (ADR 0314 / v2.71.0). Missing/non-finite U omits only that caption; finite negative and zero values remain visible. No SQL/new columns or new measurement authority.

Current exact authority

The live merge result was exercised before promotion. It preserves #854's v2.71 persisted-U delta, inherits #853's exact #852 ancestry and Vietnamese explained/unexplained/cross-share semantic repairs, keeps the parent regression files, and regenerates/verifies the editable lock with repository-authoritative uv at 2.71.0. Focused i18n regressions, lint, production build, and project lock-version parity were GREEN on the promoted workflow-free tree. This does not substitute for normal hosted PR gates, authenticated browser/a11y/performance evidence, or independent review.

Descendant convergence

Direct child #855 is still on historical #854 467b0a865a0661a275b456a1d7ea689755ec3610 and must adopt 746bd6a1... non-force before its v2.72 residual delta can be accepted. Then descendants converge in order. Parent movement is a repair finding, never a reason to close or force-rebase.

Product invariants

The v2.71 persisted-U delta remains authoritative: missing/non-finite U omits only its caption; finite zero and negative values remain explicit. Never derive U from R, , geometry, rank, coverage, counts, shares, or neighbouring statistics. Canonical KO/EN/JA/ZH/VI/ES/DE/FR translation authority remains the governed Customer Master path.

Keep out of normal merge until fresh exact-head repository/security gates, descendant convergence, authenticated browser/a11y/performance evidence, and qualifying independent approval are current. No self-approval, bypass, force-push, destructive rebase, generated-lock hand edit, ancestor skip, or stale-evidence transfer.

…on graphic (v2.71.0)

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

Copy link
Copy Markdown
Contributor Author

Copilot review requested on exact head 935cdea (v2.71.0 leftover-map unexplained leftover on grouping comparison graphic, ADR 0314). Independent APPROVE required. Copilot/Devin COMMENT is not independent APPROVE. Do not self-approve. Do not squash-merge onto the unprotected leftover base. After independent review, retarget onto leftover-coordinates (feat/leftover-map-coordinates-v2240 / #782). Next deferred caption is leftover residual R on the comparison graphic (strip already ADR 0298). Issues #79 and #87 stay open.

@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: caa851a6-5fdd-4cb5-8cce-68979f5e44e5

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.

@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

Queue retry 2026-08-31T08:01 KST: Full test suite green; Frontend lint/test/build green; Devin Review commented with no issues. Independent APPROVE still required on exact head 935cdea. Do not self-approve. Do not squash onto the unprotected leftover base. Next buyer gap is leftover residual on the grouping comparison leftover-map graphic (distinct comparison-graphic name; residual already on the strip via ADR 0298).

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 08:19 KST: exact head 935cdea has Frontend lint/test/build + Full test suite success. Devin COMMENT only — no independent APPROVE on this head. Do not self-approve. Do not squash-merge onto the unprotected leftover base. Next buyer gap is leftover residual on the grouping comparison leftover-map graphic (distinct comparison-graphic name; residual already sits on the strip via ADR 0298). Stacked PR will open onto this head after the increment lands.

Copy link
Copy Markdown
Contributor Author

Writes work this cycle. #855 (feat/leftover-map-compare-plot-residual-v2720 @ 4ffed4a, ADR 0315 / v2.72.0) is stacked on this exact head. Caption leftover residual R on the grouping comparison leftover-map graphic with distinct accessible name leftover map comparison graphic residual {label}. Leave both PRs open for independent review. Do not self-approve. Do not squash-merge.

Copy link
Copy Markdown
Contributor Author

Independent APPROVE still required on this exact head 935cdea (v2.71.0). #855 leftover residual (v2.72.0) is stacked here at 1087b16. Do not self-approve. Do not squash-merge onto the unprotected leftover base. Issues #79 and #87 stay open.

@seonghobae
seonghobae marked this pull request as draft September 4, 2026 08:06
Adopt the conflict-free GitHub test-merge tree for exact parent #853 11fc346 while preserving existing #854 product head 935cdea 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 #854 eb4ffef while preserving existing #855 product head 1087b16 as first-parent ancestry. No force push and no product delta is dropped.

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

Current-head finding on eb4ffef04677d6e1d67fe1325cfdf1bbb2db2245: this Draft PR still points to old #853 snapshot 11fc346299f9d4a96e4ba397ba373e275af2fce1, while current #853 is fab1cfd587bc6495c9d1084e86774b1378b411c6; the current compare is diverged and #854 is 22 commits behind. ADR 0314 is also marked Accepted although the PR is Draft. After the #840 foundation repair and #853 current-parent reconstruction, rebuild #854 from the current #853 while preserving its unexplained-leftover delta, and set ADR 0314 to Proposed before readiness.

Adopt exact parent #85329c3354 without rewriting #854. Preserve the #854 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
Adopt exact parent #8540014ea8 without rewriting #855. Preserve the #855 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
Preserve #855 residual-R truth and ADR 0315 Proposed while inheriting exact #854 ADR 0314 Proposed through ordinary two-parent non-force convergence.
Preserve #854 unexplained-U truth and ADR 0314 Proposed while inheriting exact #853 ADR 0313 Proposed through ordinary two-parent non-force convergence.
seonghobae added a commit that referenced this pull request Sep 10, 2026
Preserve #855 residual-R truth and Proposed ADR 0315 while inheriting repaired ADR 0313/0314 governance through ordinary two-parent non-force convergence.

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

Current-head re-review: ADR 0314 is now Proposed, and this head preserves the v2.71 persisted-U product delta while inheriting repaired ADR 0313 governance by ordinary two-parent non-force convergence. The PR remains inherited RED because #851 still owns generated-lock parity, 20px small-canvas bounds, and Vietnamese copy failures; no stale validation receipt is transferable.

seonghobae added a commit that referenced this pull request Sep 10, 2026
Preserve the v2.72 residual product delta and the already-adopted compact-canvas layout while making current #854 e34020f an explicit second parent. The resolved tree is the existing #855 exact tree, which already contains the bounded padding/caption behavior; no force rewrite or product delta removal.
seonghobae added a commit that referenced this pull request Sep 10, 2026
github-actions Bot pushed a commit that referenced this pull request Sep 10, 2026
Preserve the v2.71 persisted-U delta while adopting exact repaired #853 and regenerating the child lock with repository-authoritative uv.
Preserve the v2.71 persisted-U product delta from 467b0a8 while adopting exact repaired #853 ef02b1d as the second parent. Promote the workflow-free validated tree with v2.71 generated-lock parity and inherited Vietnamese share semantics.
@seonghobae
seonghobae marked this pull request as ready for review September 10, 2026 22:09
seonghobae added a commit that referenced this pull request Sep 10, 2026
chore(stack): converge #855 onto repaired #854
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