feat(reports): leftover observed on grouping comparison graphic (v2.73.0) - #856
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks 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.
Summary
Current stack authority
feat/leftover-map-compare-plot-residual-v2720@e8f0ca2c10584fad578c4cd7e6493025bd565020.974109a055573e3534f4e653fc9929ffad1ac8d8; PR remains Draft and now diverges from the moved parent.tests/test_adr_0315_governance.py; its ADR 0315 is intentionally RED until the parent status repair is verified and promoted. Foundation repair therefore precedes any feat(reports): leftover observed on grouping comparison graphic (v2.73.0) #856 reconstruction.5124096767found ADR 0316 prematurely declaresDecision status: Accepted;tests/test_adr_0316_governance.pyrequires Proposed while unmerged.Stale repair evidence boundary
The existing isolated #856 lane
automation/856-adr-status-repair-20260906@4bb04c897e06ca389daa2e121a30007ce4f6f5aa, run34011364534, was created from the previous #855 parent snapshot. Even if it later completes GREEN, it is not promotable after the parent moved. Do not fast-forward #856 from that tree or transfer its checks.Correct ordering is now: finish #855 ADR0315 Proposed repair on the unchanged parent head → promote its clean tree non-force → reconstruct #856 from that exact repaired #855 while preserving the v2.73/ADR0316 observed-Y delta and applying ADR0316 Proposed → rerun exact child contracts → continue to #857 and descendants. This keeps both valid product deltas and avoids rebuilding descendants on an intentional-RED foundation.
Keep Draft. Do not force-push, self-approve, weaken gates, close valid deltas, or transfer predecessor checks/approval.