feat(reports): fail-close leftover-map pair-list post coverage (v2.45.0) - #819
Conversation
Caption leftover-map post complete-case coverage on the pair list only when leftoverMapCoverageCounts returns usable complete-case integers (ADR 0288). A used-greater-than-scored, negative, or non-integer payload omits that note. UI-only; no SQL; no new columns. Never invent a leftover score. Never invent a theta.
|
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: Pro Plus 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 |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| aria-label={openedGroupingLabel ? "Opened grouping report" : "Period report groups"} | ||
| > | ||
| {orderedReports.map((report) => { | ||
| const coverageCounts = leftoverMapCoverageCounts(report.leftover_map_coverage); |
There was a problem hiding this comment.
📝 Info: Shared validation preserves coverage semantics
leftoverMapCoverageCounts rejects malformed persisted counts while preserving valid zero-used coverage. It never derives post coverage from neighboring display data.
Was this helpful? React with 👍 or 👎 to provide feedback.
| export const LEFTOVER_MAP_LIST_COVERAGE_LABEL = "Leftover map coverage"; | ||
|
|
||
| export const LEFTOVER_MAP_PLOT_COVERAGE = | ||
| "Leftover map used {used} of {scored} scored posts (complete-case)"; |
There was a problem hiding this comment.
📝 Info: Shared text keeps labels distinct
The list and plot intentionally share translated coverage text. LEFTOVER_MAP_LIST_COVERAGE_LABEL keeps their accessible names independently addressable.
Was this helpful? React with 👍 or 👎 to provide feedback.
|
Write-retry this cycle (seonghobae MCP). #819 is leftover-map v2.45 fail-close pair-list post coverage at This cycle stacks leftover-map graphic axis singular values ( |
|
Write-retry this cycle (seonghobae MCP). #819 remains leftover-map v2.45 fail-close pair-list post coverage at This cycle stacks leftover-map complete-case coverage on the grouping comparison strip as v2.46 / ADR 0289 on this head (include persisted leftover_map_coverage on GET /api/reports/compare/{period}; caption each strip row through leftoverMapCoverageCounts; distinct aria-label |
|
Stacked leftover-map graphic axis singular values ( |
|
Stacked #821 ( |
1 participant
Summary
make seed, closest and farthest leftover pairs sit above the member list with the Gabriel biplot of already-named coordinates, leftover-map axis share when finite, axis ticks that matchξ (x, y) ζ (x, y)on the pair row, pair segments that matchdon the pair row, pair segments that matchR̂on the pair row, pair segments that matchR̂²/R²on the pair row, pair segments that matchU²/R²on the pair row, pair segments that match2R̂U/R²on the pair row, pair segments that matchUon the pair row, pair segments that matchRon the pair row, pair segments that matchYon the pair row, pair segments that matchEon the pair row, pair segments that match leftover-map rank on the pair row, a plot caption that matches leftover-map complete-case coverage above the pair list, a plot caption that matches leftover-map item complete-case coverage, a plot caption that matches leftover-map incomplete post coverage, a plot caption that matches leftover-map incomplete item coverage, a pair-list note that matches leftover-map post complete-case coverage, a pair-list note that matches leftover-map item complete-case coverage, a pair-list note that matches leftover-map incomplete post coverage, and a pair-list note that matches leftover-map incomplete item coverage; click a post marker or a pair button opens that post.0 of Mis shown when that persisted used count is a non-negative integer and scored is a positive integer. Do not invent post coverage from plotted marker count, leftover-map distance, leftover-map rank, leftover-map item coverage, leftover-map incomplete post coverage, leftover-map incomplete item coverage, or the count of unused axes.feat/leftover-map-list-incomplete-item-v2440/ feat(reports): leftover-map incomplete item coverage on pair list (v2.44.0) #818 @aa32089). Leave this PR open for independent review. Do not squash-merge onto the unprotected leftover base. Do not self-approve. After independent review, retarget onto leftover-coordinates (feat/leftover-map-coordinates-v2240/ feat(reports): persist leftover-map coordinates (v2.24.0) #782) rather than squash-merging here.Verification
leftoverMapCoverageCountsalready omits used-greater-than-scored, negative, and non-integer counts. Pair-list labelLeftover map coverageis distinct from graphic labelLeftover-map graphic coverage.tests/test_leftover_pairs.pyand PostgreSQL period-report tests were not executed here (UI-only; no schema change)Evidence boundary
Authenticated PostgreSQL API and k6 runtime acceptance are not claimed. No real records or credentials. Synthetic fixtures only.
Independent approval is required. No self-approval, admin bypass, or force push.
Jeon, M., Jin, I. H., Schweinberger, M., & Baugh, S. (2021). Mapping unobserved item–respondent interactions: A latent space item response model with interaction map. Psychometrika, 86(2), 378–403.
Gabriel, K. R. (1971). The biplot graphic display of matrices with application to principal component analysis. Biometrika, 58(3), 453–467.