feat(reports): leftover observed on grouping comparison graphic (v2.73.0) - #856
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
|
Cycle 2026-08-31 08:52 KST: exact head |
1a2f11c to
c7a8680
Compare
|
Exact head now |
…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.
c7a8680 to
cf3b6d4
Compare
|
Rebased onto #855 @ Comparison-graphic leftover observed uses distinct accessible name 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 |
| aria-label={tf( | ||
| variant === "comparison" | ||
| ? LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_OBSERVED | ||
| : LEFTOVER_MAP_PLOT_SEGMENT_OBSERVED, | ||
| { label: segment.observedLabel }, | ||
| )} |
There was a problem hiding this comment.
| aria-label={tf( | ||
| variant === "comparison" | ||
| ? LEFTOVER_MAP_COMPARE_PLOT_SEGMENT_OBSERVED | ||
| : LEFTOVER_MAP_PLOT_SEGMENT_OBSERVED, | ||
| { label: segment.observedLabel }, | ||
| )} |
|
Cycle 2026-08-31 08:19 KST: exact head |
seonghobae
left a comment
There was a problem hiding this comment.
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.
Preserve #856 observed-Y truth and ADR 0316 Proposed while inheriting repaired ADR 0314/0315 governance through ordinary two-parent non-force convergence.
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.
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
left a comment
There was a problem hiding this comment.
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.
Summary
Caption persisted leftover observed
Yon the grouping-comparison leftover-map graphic (ADR 0316 / v2.73.0) without inventing scores or columns. Missing/non-finiteYomits only this caption; finite negativeYremains visible.Current exact authority
701c5ba6dde80ffa19a78e57d6321b29328aa1ee87de090ead8a39fdf0d12bc1ca5b5c6d3b5ebad0Proposed; predecessor acceptance does not transfer across head movementConvergence 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.