Skip to content

feat(reports): leftover-map reconstruction on grouping comparison graphic (v2.67.0) - #850

Draft
seonghobae wants to merge 4 commits into
feat/leftover-map-compare-plot-incomplete-item-v2660from
feat/leftover-map-compare-plot-reconstruction-v2670
Draft

feat(reports): leftover-map reconstruction on grouping comparison graphic (v2.67.0)#850
seonghobae wants to merge 4 commits into
feat/leftover-map-compare-plot-incomplete-item-v2660from
feat/leftover-map-compare-plot-reconstruction-v2670

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Caption persisted leftover-map reconstruction on the grouping-comparison leftover-map graphic when formatLeftoverMapReconstruction returns a usable signed badge (ADR 0310 / v2.67.0). Reconstruction omits independently from distance and coverage notes. No SQL/new columns; never reconstruct from coordinates, distance, residual, rank, coverage, pair count, or axis count.

Live authority

  • parent/base: #849@b83479b7063f3a3a7e880ed6869bb940bb470508
  • current product head: 40a6b04c1a1fe81dbb6297aadea9158b9b644955
  • PR: open / Draft

Parent convergence

After retargeting to current #849, #850 inherited the ADR-governance regression as 3e67473419b0e663833fd82208588bc7e3457426. GitHub then generated current-parent merge f6b81588a4b216bf02ed7b60178d3a19ee938534, verified with parents #849@b83479b7... and #850@3e674734.... Its tree preserves ADR 0310/v2.67.0 while carrying ADR 0304 Proposed and the regression. The same tree was recorded child-first as 40a6b04c...; branch movement was non-force only.

Historical 790 frontend tests/oxlint/py_compile evidence predates this exact head and is not transferred. Exact-head required/frontend/browser/a11y and PostgreSQL/k6 evidence plus independent approval remain outstanding. Any direct descendant must be reconstructed before predecessor evidence transfers.

Product invariants

  • Missing/non-finite omits only the comparison-graphic reconstruction caption and preserves usable distance/coverage captions.
  • Finite zero/negative reconstruction is displayed, not clamped.
  • This increment does not add explained-share captioning.

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

…phic (v2.67.0)

Caption persisted leftover-map reconstruction R̂ on the grouping comparison
leftover-map graphic from already-named leftover_map_reconstruction (ADR 0310).
Comparison copy uses leftover map comparison graphic reconstruction {label}
so it stays distinct from hyphen leftover-map reconstruction {label} and from
strip Leftover map comparison reconstruction. A missing or non-finite R̂ omits
that caption and keeps leftover-map distance and comparison graphic coverage
notes. Rank-0 origin cells still name R̂ 0.00 when that reconstruction is
finite. Do not invent R̂ from leftover-map distance or plotted coordinates.
Never invent a leftover score. Never invent a theta. No new columns.
@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.

@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 +313 to +318
aria-label={tf(
variant === "comparison"
? LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_RECONSTRUCTION
: LEFTOVER_MAP_PLOT_SEGMENT_RECONSTRUCTION,
{ label: segment.reconstructionLabel },
)}

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: Report labels remain unchanged

The variant branch changes only comparison accessibility copy. Report plots retain the existing template and both variants use the same validated value.

Devin Review

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

export const LEFTOVER_MAP_PLOT_SEGMENT_RECONSTRUCTION =
"leftover-map reconstruction {label}";

export { LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_RECONSTRUCTION } from "./leftoverMapReconstruction";

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: Comparison copy has one source

The layout module re-exports the reconstruction label instead of duplicating it. Formatter and component callers therefore share one constant.

Devin Review

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

Copy link
Copy Markdown
Contributor Author

Queue retry: leftover-map reconstruction on grouping comparison graphic (v2.67.0 / ADR 0310). Independent APPROVE required before squash-merge. Do not self-approve. Next stacked increment is leftover-map explained leftover share on that comparison graphic (ADR 0311 / v2.68.0). Do not caption unexplained leftover share this next increment. Issues #79 and #87 stay open.

Copy link
Copy Markdown
Contributor Author

Cycle retry 2026-08-31 06:25 KST: exact head 85451d8 still has no independent APPROVE (Devin COMMENT only). Leave #850 open. Do not self-approve. Do not squash-merge onto the unprotected leftover base. After independent review, retarget onto leftover-coordinates (#782, already on main) rather than squash-merging here.

Next stacked increment (ADR 0311 / v2.68.0) captions leftover-map explained leftover share on the grouping comparison leftover-map graphic with distinct accessible name leftover map comparison graphic explained leftover share {label}. It does not caption unexplained leftover share this increment. Stack onto this PR; leave that PR open for independent review.

Adopt the conflict-free GitHub test-merge tree for exact parent #849 223e988 while preserving existing #850 product head 85451d8 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 #850 e14d7e0 while preserving existing #851 product head 2d6e8e7 as first-parent ancestry. No force push and no product delta is dropped.
Adopt the GitHub-verified conflict-free merge tree for exact parent #849b83479b and child 3e67473. The tree preserves ADR 0310/v2.67.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 #85040a6b04 and child 74e22f0. The tree preserves ADR 0311/v2.68.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