Skip to content

feat(reports): leftover observed on grouping comparison graphic (v2.73.0) - #856

Draft
seonghobae wants to merge 13 commits into
feat/leftover-map-compare-plot-residual-v2720from
feat/leftover-map-compare-plot-observed-v2730
Draft

feat(reports): leftover observed on grouping comparison graphic (v2.73.0)#856
seonghobae wants to merge 13 commits into
feat/leftover-map-compare-plot-residual-v2720from
feat/leftover-map-compare-plot-observed-v2730

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Caption persisted leftover observed Y on the grouping-comparison leftover-map graphic (ADR 0316 / v2.73.0) without inventing scores or columns. Missing/non-finite Y omits only this caption; finite negative Y remains visible.

Current exact authority

Convergence PR #1004 merged normally into the child branch, preserving the v2.73 observed-Y product delta while adopting the repaired #855#854#853 ancestry. Fresh exact-head repository/security/browser/a11y/performance evidence and qualifying independent approval remain required. Direct child #857 has been converged from this exact head through #1005.

No self-approval, bypass, force-push, destructive rebase, generated-lock hand edit, ancestor skip, or stale-evidence transfer.

@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: 5b8efd22-abda-49f1-b34f-dfc3da731844

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

Cycle 2026-08-31 08:52 KST: exact head 1a2f11cf (v2.73.0 / ADR 0316 leftover observed on grouping comparison leftover-map graphic). Stacked on #855 @ 4ffed4ae. Independent APPROVE required. Do not self-approve. Do not squash-merge without independent APPROVE + exact-head required success. Copilot review requested this cycle. Next buyer increment: leftover expected E on that comparison graphic (ADR 0317 / v2.74.0).

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

Note

This report is out of date. Scroll down for Devin Review's latest report on this PR.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@seonghobae
seonghobae force-pushed the feat/leftover-map-compare-plot-observed-v2730 branch from 1a2f11c to c7a8680 Compare August 30, 2026 23:55

Copy link
Copy Markdown
Contributor Author

Exact head now c7a8680 stacked on #855 @ 1087b16 (v2.72.0 leftover residual a11y name). Comparison graphic leftover observed uses leftover map comparison graphic leftover observed {label} so it stays distinct from hyphen leftover observed {label} and strip Leftover map comparison observed. Independent APPROVE still required on this leftover-observed increment before squash-merge. Do not self-approve. Do not squash-merge onto the unprotected leftover base. Issues #79 and #87 stay open. Next buyer increment is leftover expected E on this comparison graphic (ADR 0317 / v2.74.0), stacked here — not mixed into #74/#92.

…3.0)

Name leftover observed Y on grouping comparison leftover-map pair
segments (ADR 0316). After make seed, closest and farthest leftover
pairs sit above the member list with leftover observed Y next to
leftover residual R on that comparison graphic; click opens that post.
Rank-0 unused axes still name Y 0.00 when that persisted leftover
observed is finite. A missing or non-finite Y omits that leftover-map
comparison graphic observed caption and keeps leftover residual R.
Never invent a leftover score. Never invent a theta.
@seonghobae
seonghobae force-pushed the feat/leftover-map-compare-plot-observed-v2730 branch from c7a8680 to cf3b6d4 Compare August 31, 2026 00:00

Copy link
Copy Markdown
Contributor Author

Rebased onto #855 @ 1087b16 after the leftover residual a11y rename. Head is now cf3b6d4.

Comparison-graphic leftover observed uses distinct accessible name leftover map comparison graphic leftover observed {label} (same leftover-qualifier pattern as residual). Vietnamese copy names the leftover map. Local oxlint 0; 502 tests passed (App 103, LeftoverMapPlot 55, leftoverMapPlotLayout 58, i18n 272, leftoverObservedExpected 9, leftoverResidual 5).

Leave open for independent review. Do not self-approve. Do not squash-merge without independent APPROVE + exact-head required success. Next buyer increment after this residual: leftover expected E on the comparison graphic (ADR 0317 / v2.74.0).

Copy link
Copy Markdown
Contributor Author

Independent APPROVE still required on this leftover-observed head c7a8680 (v2.73.0). #857 leftover expected (v2.74.0) is stacked here at 9f34d2c. Do not self-approve. Do not squash-merge onto the unprotected leftover base. Issues #79 and #87 stay open.

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

Devin Review

Comment on lines +433 to +438
aria-label={tf(
variant === "comparison"
? LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_OBSERVED
: LEFTOVER_MAP_PLOT_SEGMENT_OBSERVED,
{ label: segment.observedLabel },
)}

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: Observed values remain data-backed

observedLabel still uses persisted observed_response through its finite-value formatter. The variant branch changes only the accessible name, preserving independent omission.

Devin Review

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

Comment on lines +433 to +438
aria-label={tf(
variant === "comparison"
? LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_OBSERVED
: LEFTOVER_MAP_PLOT_SEGMENT_OBSERVED,
{ label: segment.observedLabel },
)}

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: Plot variants keep separate labels

The comparison branch receives the new translation key. Report plots retain LEFTOVER_MAP_PLOT_SEGMENT_OBSERVED, so their accessible contract stays unchanged.

Devin Review

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

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 08:19 KST: exact head cf3b6d4 (v2.73.0 / ADR 0316 leftover observed on grouping comparison leftover-map graphic). Independent APPROVE still required on this leftover-map stack before any squash-merge. Do not self-approve. Issues #79 and #87 stay open. Do not merge #808 without independent APPROVE. Next buyer increment is leftover-map rank on this comparison graphic after leftover expected (#857 / ADR 0317 / v2.74.0).

@seonghobae
seonghobae marked this pull request as draft September 4, 2026 08:05
Adopt the conflict-free GitHub test-merge tree for exact parent #855 e38c7ca while preserving existing #856 product head cf3b6d4 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 #856 dbb3079 while preserving existing #857 product head ade9ed5 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 repair finding: ADR 0316 is introduced by this still-Draft PR as Decision status: Accepted. That is premature decision authority before qualifying independent review/normal merge. Preserve the observed-caption product/UI delta, but keep ADR 0316 Proposed while this PR is Draft and add an executable governance regression. The PR body is also stale: live base is #855 at e38c7caf45ed51825eba73047f78052e8d51a431 and live head is dbb30794bdf5af1ae85f2e9e1ea39ba434e9bd14, not the abbreviated older Stack line. No force-push, self-approval, or gate weakening.

Adopt exact parent #855177c6f5 without rewriting #856. Preserve the #856 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 #856e34f32c without rewriting #857. Preserve the #857 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 the #857 expected-E product delta and ADR 0317 Proposed repair while adopting exact #856 ADR 0316 Proposed by ordinary two-parent non-force convergence.
Preserve the #856 observed-Y product delta and ADR 0316 Proposed repair while adopting exact #855 ADR 0315 Proposed by ordinary two-parent non-force convergence.
seonghobae added a commit that referenced this pull request Sep 10, 2026
Preserve #857 expected-E truth and ADR 0317 Proposed while inheriting #855/#856 Proposed governance through ordinary two-parent non-force convergence.
Preserve #856 observed-Y truth and ADR 0316 Proposed while inheriting repaired ADR 0314/0315 governance through ordinary two-parent non-force convergence.
seonghobae added a commit that referenced this pull request Sep 10, 2026
Preserve #857 expected-E truth and ADR 0317 Proposed while inheriting repaired ADR 0314/0315/0316 governance through ordinary two-parent non-force convergence.
Preserve #856 observed-Y truth and Proposed ADR 0316 while inheriting repaired ADR 0313/0314/0315 governance through ordinary two-parent non-force convergence.
seonghobae added a commit that referenced this pull request Sep 10, 2026
Preserve #857 expected-E truth and Proposed ADR 0317 while inheriting repaired ADR 0313/0314/0315/0316 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 0316 is Proposed and the observed-Y product delta remains intact after non-force convergence through repaired ADR 0313–0315. This is not an acceptance review: #851's generated-lock and UI/i18n REDs still gate the stack, so fresh exact-head evidence is required after the parent is causally GREEN.

Preserve the v2.73 observed-Y product delta and ADR 0316 Proposed while inheriting #855 db9871e as an explicit parent. Semantic resolution applies the already-GREEN compact-canvas layout and keeps the distinct grouping-comparison observed label export. No force rewrite or product delta removal.
seonghobae added a commit that referenced this pull request Sep 10, 2026
Preserve the v2.74 expected-E product delta and ADR 0317 Proposed while inheriting #856 0ff6dee as an explicit parent. Semantic resolution keeps the already-GREEN compact-canvas layout and both distinct comparison observed/expected label exports. No force rewrite or product delta removal.
@seonghobae
seonghobae marked this pull request as ready for review September 10, 2026 22:11
chore(stack): converge #856 onto repaired #855
seonghobae added a commit that referenced this pull request Sep 10, 2026
chore(stack): converge #857 onto repaired #856
@seonghobae
seonghobae marked this pull request as draft September 11, 2026 20:19
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