From 772928722264b001c669b580d19866ccd8b590e0 Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Sun, 30 Aug 2026 11:29:15 +0000 Subject: [PATCH 1/7] feat(reports): leftover-map incomplete post coverage on graphic display (v2.40.0) Caption the leftover-map graphic display with persisted leftover-map incomplete post coverage (ADR 0283). After make seed, closest and farthest leftover pairs sit above the member list with a plot caption that matches leftover-map incomplete post coverage; click a post marker or a pair button opens that post. A missing, non-integer, or negative dropped count, or a dropped count that contradicts usable complete-case integers, omits that leftover-map incomplete post caption. Dropped 0 is shown when that persisted dropped count is a non-negative integer. Do not invent dropped posts from scored minus used, plotted marker count, leftover-map distance, leftover-map rank, leftover-map post coverage, leftover-map item coverage, or the count of unused axes. Never invent a leftover score or a theta. UI-only; no new columns. --- AGENTS.md | 11 +- ARCHITECTURE.md | 8 +- .../2.40.0-leftover-map-plot-incomplete.md | 26 +++ CHANGELOG.md | 25 +++ CLAUDE.md | 2 +- docs/adr/0283-leftover-map-plot-incomplete.md | 166 ++++++++++++++++++ docs/product-technical-gap-baseline.md | 51 ++++++ frontend/package.json | 2 +- frontend/src/App.test.tsx | 3 + frontend/src/components/LeftoverMapPlot.css | 3 +- .../src/components/LeftoverMapPlot.test.tsx | 62 ++++++- frontend/src/components/LeftoverMapPlot.tsx | 21 ++- .../src/components/LeftoverPairList.test.tsx | 3 + frontend/src/components/LeftoverPairList.tsx | 3 +- frontend/src/i18n.test.ts | 24 ++- frontend/src/i18n.ts | 28 ++- frontend/src/leftoverMapCoverage.test.ts | 41 ++++- frontend/src/leftoverMapCoverage.ts | 31 +++- frontend/src/leftoverMapPlotLayout.ts | 6 +- pyproject.toml | 2 +- 20 files changed, 485 insertions(+), 33 deletions(-) create mode 100644 CHANGELOG.d/2.40.0-leftover-map-plot-incomplete.md create mode 100644 docs/adr/0283-leftover-map-plot-incomplete.md diff --git a/AGENTS.md b/AGENTS.md index 2e7d2fe57..a5d5d9996 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -273,7 +273,7 @@ stops startup instead of leaving a healthy-looking partial schema, and application code must not compensate for a missing table. Period leftover pairs (ADR 0017 / 0018 / 0048 / 0049 / 0119 / 0158 / 0162 / -0163 / 0164 / 0182 / 0185 / 0201 / 0233 / 0266 / 0267 / 0268 / 0269 / 0270 / 0271 / 0272 / 0273 / 0274 / 0275 / 0276 / 0277 / 0278 / 0279 / 0280 / 0281 / 0282) are computed in `lineageweave/leftover_pairs.py` from the +0163 / 0164 / 0182 / 0185 / 0201 / 0233 / 0266 / 0267 / 0268 / 0269 / 0270 / 0271 / 0272 / 0273 / 0274 / 0275 / 0276 / 0277 / 0278 / 0279 / 0280 / 0281 / 0282 / 0283) are computed in `lineageweave/leftover_pairs.py` from the residual after a real GRM/GPCM score, never invented. Distances are Euclidean on the two-dimensional Gabriel leftover map; missing cells stay out of the factorization. Closest and farthest post–criterion pairs @@ -348,6 +348,12 @@ coverage so two criterion diamonds are not read as the scored-criterion census; a missing, non-integer, negative-used, non-positive-scored, or used-greater-than-scored item coverage omits that leftover-map item coverage caption and does not invent item coverage from plotted criterion marker count. +ADR 0283 captions the leftover-map graphic display with persisted leftover-map +incomplete post coverage so a `used N of M` caption is not read as every +scored post entering the map; a missing, non-integer, or negative dropped +count, or a dropped count that contradicts usable complete-case integers, +omits that leftover-map incomplete post caption and does not invent dropped +posts from scored minus used or from plotted marker count. When `R`, `R̂`, `U`, `x`, `s`, and `e` are finite, `e + s + x = 1`. When `Y`, `E`, and `R` are finite, `Y − E = R`. When `R`, `R̂`, and `U` are @@ -362,7 +368,8 @@ and are not a leftover score. Complete-case coverage (ADR 0168) persists to `report_leftover_map_coverage` and captions the pair list and the leftover-map graphic with how many scored posts entered the map. Item complete-case coverage (ADR 0282) captions the leftover-map graphic with how many scored criteria -entered the map. +entered the map. Incomplete post coverage (ADR 0283) captions the leftover-map +graphic with how many scored posts stayed out of the factorization. Authorized occupational construct catalog search (ADR 0257) matches official O*NET preferred labels or descriptions only when a source-eligible, ABAC-visible diff --git a/ARCHITECTURE.md b/ARCHITECTURE.md index d1e13c158..e38514fdd 100644 --- a/ARCHITECTURE.md +++ b/ARCHITECTURE.md @@ -637,7 +637,7 @@ information at the group's mean θ (Lord, 1980 max-info CAT). Rankings persist to `report_item_information`. After those IRT main effects, residual SVD leftover pairs on two Gabriel axes (Jeon et al., 2021; ADR 0017 / 0048 / 0049 / 0119 / 0148 / 0158 / 0162 / 0163 / 0164 / 0168 / -0182 / 0185 / 0201 / 0233 / 0266 / 0267 / 0268 / 0269 / 0270 / 0271 / 0272 / 0273 / 0274 / 0275 / 0276 / 0277 / 0278 / 0279 / 0280 / 0281 / 0282) persist to `report_leftover_pair` with signed residual `R`, +0182 / 0185 / 0201 / 0233 / 0266 / 0267 / 0268 / 0269 / 0270 / 0271 / 0272 / 0273 / 0274 / 0275 / 0276 / 0277 / 0278 / 0279 / 0280 / 0281 / 0282 / 0283) persist to `report_leftover_pair` with signed residual `R`, observed `Y`, expected `E[Y|θ, item]`, full leftover-map rank, unexplained leftover, ADR 0201 reconstruction evidence, ADR 0185 cross-share evidence, ADR 0233 unexplained leftover share `s`, ADR 0266 explained leftover @@ -660,14 +660,16 @@ observed `Y`. ADR 0279 captions leftover-map pair segments with persisted leftov expected `E`. ADR 0280 captions leftover-map pair segments with persisted leftover-map rank. ADR 0281 captions the leftover-map graphic display with persisted leftover-map complete-case coverage. ADR 0282 captions the leftover-map graphic display with -persisted leftover-map item complete-case coverage. Click a post marker or a pair button +persisted leftover-map item complete-case coverage. ADR 0283 captions the leftover-map +graphic display with persisted leftover-map incomplete post coverage. Click a post marker or a pair button opens that post. Those ADRs are the normative mathematical, storage, and display contracts. Leftover-map axis share (Gabriel inertia of residual SVD axes 1 and 2; ADR 0148) persists to `report_leftover_map_axis`. Complete-case leftover-map coverage (ADR 0168) persists to `report_leftover_map_coverage` so readers see how many scored posts entered the factorization, including on the graphic, -and how many scored criteria entered the factorization on the graphic. Results persist to +and how many scored criteria entered the factorization on the graphic, +and how many scored posts stayed incomplete and out of the factorization. Results persist to `report_period_score` / `report_member_score`. `GET /api/reports/{grouping}` lists the trend; `GET /api/reports/{grouping}/{period}` is ABAC-filtered; diff --git a/CHANGELOG.d/2.40.0-leftover-map-plot-incomplete.md b/CHANGELOG.d/2.40.0-leftover-map-plot-incomplete.md new file mode 100644 index 000000000..612ea9fa2 --- /dev/null +++ b/CHANGELOG.d/2.40.0-leftover-map-plot-incomplete.md @@ -0,0 +1,26 @@ +## 2.40.0 — Leftover-map incomplete post coverage on the graphic display + +- Caption the leftover-map graphic display with persisted leftover-map + incomplete post coverage (ADR 0283). After `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 match `d` on the pair row, pair segments that match `R̂` on the + pair row, pair segments that match `R̂²/R²` on the pair row, pair + segments that match `U²/R²` on the pair row, pair segments that match + `2R̂U/R²` on the pair row, pair segments that match `U` on the pair + row, pair segments that match `R` on the pair row, pair segments that + match `Y` on the pair row, pair segments that match `E` on 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, and a plot caption that matches leftover-map incomplete post + coverage; click a post marker or a pair button opens that post. + A missing, non-integer, or negative dropped count, or a dropped count + that contradicts usable complete-case integers, omits that leftover-map + incomplete post caption. Dropped `0` is shown when that persisted + dropped count is a non-negative integer. Do not invent dropped posts + from scored minus used, plotted marker count, leftover-map distance, + leftover-map rank, leftover-map post coverage, leftover-map item + coverage, or the count of unused axes. Never invent a leftover score. + Never invent a theta. No new columns. diff --git a/CHANGELOG.md b/CHANGELOG.md index 18a6a0baf..df3f58696 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,6 +8,31 @@ All notable changes to this project are documented here. Format follows ### Added +- Period leftover pairs now caption the leftover-map graphic display with + persisted leftover-map incomplete post coverage (ADR 0283 / v2.40.0). After + `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 match `d` on the pair row, pair + segments that match `R̂` on the pair row, pair segments that match + `R̂²/R²` on the pair row, pair segments that match `U²/R²` on the pair + row, pair segments that match `2R̂U/R²` on the pair row, pair segments + that match `U` on the pair row, pair segments that match `R` on the + pair row, pair segments that match `Y` on the pair row, pair segments + that match `E` on 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, and a plot caption that + matches leftover-map incomplete post coverage; click a post marker or a + pair button opens that post. A missing, non-integer, or negative + dropped count, or a dropped count that contradicts usable complete-case + integers, omits that leftover-map incomplete post caption. Dropped `0` + is shown when that persisted dropped count is a non-negative integer. + Do not invent dropped posts from scored minus used, plotted marker + count, leftover-map distance, leftover-map rank, leftover-map post + coverage, leftover-map item coverage, or the count of unused axes. + Never invent a leftover score or a theta. No new columns. + - Period leftover pairs now caption the leftover-map graphic display with persisted leftover-map item complete-case coverage (ADR 0282 / v2.39.0). After `make seed`, closest and farthest leftover pairs sit above the member diff --git a/CLAUDE.md b/CLAUDE.md index a342ef2c6..70e509a8a 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -49,7 +49,7 @@ cutoff. Global Ask optional `knowledge_cutoff` uses the same cover Create/start endpoint rules (ADR 0017 / 0021), tie-vs-miss similarity (ADR 0026), R&R catalog ids (ADR 0019 / 0027), leftover pairs -(ADR 0048–0164 / 0182 / 0185 / 0201 / 0233 / 0266 / 0267 / 0268 / 0269 / 0270 / 0271 / 0272 / 0273 / 0274 / 0275 / 0276 / 0277 / 0278 / 0279 / 0280 / 0281 / 0282), occupational construct catalog search +(ADR 0048–0164 / 0182 / 0185 / 0201 / 0233 / 0266 / 0267 / 0268 / 0269 / 0270 / 0271 / 0272 / 0273 / 0274 / 0275 / 0276 / 0277 / 0278 / 0279 / 0280 / 0281 / 0282 / 0283), occupational construct catalog search (ADR 0257), the text-channel embedding swap and cosine clamp (ADR 0190), per-edge channel-score persistence (ADR 0195), token-backed status notices (ADR 0220), diff --git a/docs/adr/0283-leftover-map-plot-incomplete.md b/docs/adr/0283-leftover-map-plot-incomplete.md new file mode 100644 index 000000000..46cc00ef4 --- /dev/null +++ b/docs/adr/0283-leftover-map-plot-incomplete.md @@ -0,0 +1,166 @@ +# ADR 0283 — Name leftover-map incomplete post coverage on the graphic display + +**Decision status:** Accepted +**Date:** 2026-08-30 + +Amends [ADR 0268](0268-leftover-map-graphic-display.md), +[ADR 0049](0049-leftover-pair-report-ui.md), and leftover-map complete-case +coverage ([ADR 0168](0168-leftover-map-complete-case-coverage.md)). Independent of +leftover-map item complete-case coverage on the graphic display +([ADR 0282](0282-leftover-map-plot-item-coverage.md)), leftover-map complete-case +coverage on the graphic display +([ADR 0281](0281-leftover-map-plot-coverage.md)), leftover-map rank on pair +segments ([ADR 0280](0280-leftover-map-segment-rank.md)), leftover expected on pair +segments ([ADR 0279](0279-leftover-map-segment-expected.md)), leftover observed on +pair segments ([ADR 0278](0278-leftover-map-segment-observed.md)), leftover +residual on pair segments ([ADR 0277](0277-leftover-map-segment-residual.md)), +leftover-map unexplained leftover on pair segments +([ADR 0276](0276-leftover-map-segment-unexplained-leftover.md)), leftover-map +cross share on pair segments ([ADR 0275](0275-leftover-map-segment-cross-share.md)), +leftover-map unexplained leftover share on pair segments +([ADR 0274](0274-leftover-map-segment-unexplained-share.md)), leftover-map +explained leftover share on pair segments +([ADR 0273](0273-leftover-map-segment-explained-share.md)), leftover-map +reconstruction on pair segments +([ADR 0272](0272-leftover-map-segment-reconstruction.md)), leftover-map +distance on pair segments ([ADR 0271](0271-leftover-map-segment-distance.md)), +leftover-map coordinate ticks ([ADR 0270](0270-leftover-map-coordinate-ticks.md)), +leftover-map axis share on the graphic display +([ADR 0269](0269-leftover-map-axis-share-plot.md)), leftover residual +disclosure ([ADR 0162](0162-leftover-residual-disclosure.md)), leftover +observed `Y` / expected `E` ([ADR 0163](0163-leftover-observed-expected.md)), +leftover-map explained leftover share persistence +([ADR 0266](0266-leftover-map-explained-share.md)), leftover-map +unexplained leftover share persistence +([ADR 0233](0233-leftover-map-unexplained-share.md)), leftover-map +reconstruction persistence ([ADR 0201](0201-leftover-map-reconstruction.md)), +leftover-map cross share persistence +([ADR 0185](0185-leftover-map-cross-share.md)), leftover-map unexplained leftover +persistence ([ADR 0182](0182-leftover-map-unexplained.md)), leftover-map rank +persistence ([ADR 0164](0164-leftover-map-rank.md)), and leftover-map axis +share persistence ([ADR 0148](0148-leftover-map-axis-share.md)). + +## Context + +ADR 0168 already persists `incomplete_post_count` on +`report_leftover_map_coverage`. ADR 0281 already names persisted leftover-map +post complete-case coverage on the leftover-map graphic display. ADR 0282 +already names persisted leftover-map item complete-case coverage on that +graphic. The graphic still has no incomplete-post caption, so a buyer who +reads `used N of M scored posts (complete-case)` can treat those N posts as +the full scored census even after incomplete rows were dropped. Incomplete +rows are dropped from Gabriel factorization; hiding dropped posts on the plot +lets a sparse post with one missing criterion vanish without a next action. + +This increment names leftover-map incomplete post coverage on the +leftover-map graphic display. It does not add columns. It does not recompute +dropped posts from scored minus used, plotted marker count, leftover-map +distance, or leftover-map rank. It does not persist leftover-map inner +product, cosine, or length as separate columns. It does not land Post quality +on the leftover criterion. Leftover-map distance stays two-axis Euclidean. Do +not invent a leftover score. Do not invent a theta. + +The dashboard stack already used neighbouring leftover facts under other +numbers. This protected increment uses **0283** so it does not collide with +leftover-map item complete-case coverage on the graphic display (0282), +leftover-map complete-case coverage on the graphic display (0281), leftover-map +rank on pair segments (0280), leftover expected on pair segments (0279), leftover +observed on pair segments (0278), leftover residual on pair segments (0277), +leftover-map unexplained leftover on pair segments (0276), leftover-map cross +share on pair segments (0275), leftover-map unexplained leftover share on pair +segments (0274), leftover-map explained leftover share on pair segments (0273), +leftover-map reconstruction on pair segments (0272), leftover-map distance on +pair segments (0271), leftover-map coordinate ticks (0270), leftover-map axis +share on the graphic display (0269), leftover-map graphic display (0268), +leftover-map coordinates (0267 / migration 0245), leftover-map explained leftover +share persistence (0266 / migration 0244), leftover-map unexplained leftover +share persistence (0233 / migration 0233), leftover-map reconstruction +persistence (0201 / migration 0206), leftover-map cross share persistence (0185), +leftover-map unexplained leftover persistence (0182), leftover residual +disclosure (0162), leftover observed `Y` / expected `E` persistence (0163), +leftover-map rank persistence (0164), leftover coverage persistence (0168), +two-axis leftover-map distance persistence, leftover-map axis share persistence +(0148), leftover interaction-map persistence, occupational construct catalog +search (0265), or the dashboard stacks. + +## Decision + +On the leftover-map graphic display, caption persisted leftover-map incomplete +post coverage as `Leftover map dropped {dropped} incomplete posts`, using the +persisted `incomplete_post_count` integer. A missing coverage row, a +non-integer dropped count, a negative dropped count, or a dropped count that +contradicts usable complete-case integers (`dropped !== scored − used` when +`map_post_count` / `scored_post_count` are usable) omits that leftover-map +incomplete post caption and keeps the connecting lines and any leftover-map +distance, reconstruction, explained leftover share, unexplained leftover share, +leftover-map cross share, unexplained leftover, leftover residual, leftover +observed, leftover expected, leftover-map rank, leftover-map post coverage, or +leftover-map item coverage caption. Dropped `0` is shown when that persisted +dropped count is a non-negative integer. Do not invent dropped posts from +scored minus used, plotted marker count, leftover-map distance, leftover-map +rank, leftover-map post coverage, leftover-map item coverage, or the count of +unused axes. Click a post marker to open that post. Criterion markers are not +post buttons. The grouping comparison strip (ADR 0149) stays on its reduced +leftover payload and does not gain this leftover-map incomplete post caption. +The pair-list note (ADR 0168) stays post coverage. + +Do not add SQL. Do not edit shipped migrations. Do not persist inner +product, cosine, or length as separate columns. Do not invent a leftover +score. Do not invent a theta. + +## Consequences + +After `make seed`, closest and farthest leftover pairs sit above the member +list with the leftover-map graphic display of persisted `ξ` and `ζ`, +leftover-map axes name persisted Gabriel inertia share when finite, +leftover-map axis ticks name the same coordinates shown on the pair row, +pair segments name persisted leftover-map distance `d`, persisted leftover-map +reconstruction `R̂`, persisted leftover-map explained leftover share `e`, +persisted leftover-map unexplained leftover share `s`, persisted leftover-map +cross share `x`, persisted leftover-map unexplained leftover `U`, persisted +leftover residual `R`, persisted leftover observed `Y`, persisted leftover +expected `E`, persisted leftover-map rank, the plot names persisted leftover-map +complete-case coverage, the plot names persisted leftover-map item +complete-case coverage, and the plot names persisted leftover-map incomplete +post coverage; click a post marker or a pair button opens that post. +Hidden posts stay hidden. Rank-0 unused axes still plot at the origin and still +name incomplete posts when that dropped count is persisted. + +## Related + +Independent of leftover interaction-map persistence, leftover-criterion +evaluation landing, leftover residual disclosure, leftover-map +complete-case coverage persistence, leftover-map axis share persistence, +leftover pairs on the grouping comparison strip, two-axis leftover-map +distance persistence, leftover-map rank persistence, leftover-map inner +product, leftover-map cosine, leftover-map length, leftover-map +reconstruction persistence, leftover-map unexplained leftover persistence, +leftover-map cross share persistence, leftover-map unexplained leftover +share persistence, leftover-map explained leftover share persistence, +leftover-map coordinate persistence, leftover-map graphic display, +leftover-map axis share on the graphic display, leftover-map coordinate +ticks, leftover-map distance on pair segments, leftover-map reconstruction +on pair segments, leftover-map explained leftover share on pair segments, +leftover-map unexplained leftover share on pair segments, leftover-map +cross share on pair segments, leftover-map unexplained leftover on pair +segments, leftover residual on pair segments, leftover observed on pair +segments, leftover expected on pair segments, leftover-map rank on pair +segments, leftover-map complete-case coverage on the graphic display, and +leftover-map item complete-case coverage on the graphic display. + +## References + +Gabriel, K. R. (1971). The biplot graphic display of matrices with +application to principal component analysis. *Biometrika, 58*(3), +453–467. https://doi.org/10.1093/biomet/58.3.453 + +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. +https://doi.org/10.1007/s11336-021-09762-5 +(LSIRM interaction `−γ‖ξ_j − ζ_i‖` after main effects +`α_j − β_i`; typically `p = 2` for the interaction map. Gabriel +reconstruction of the leftover cell is the two-axis inner product +`R̂ = ξ_{1:2} · ζ_{1:2}`. Incomplete rows are dropped from the +complete-case residual rectangle; incomplete post coverage names how many +scored posts stayed out of that factorization.) diff --git a/docs/product-technical-gap-baseline.md b/docs/product-technical-gap-baseline.md index 4370dfe60..583a02c33 100644 --- a/docs/product-technical-gap-baseline.md +++ b/docs/product-technical-gap-baseline.md @@ -1,5 +1,56 @@ # Product & Technical Gap Baseline +> Exact-head loop overlay: 2026-08-30 20:20 KST. Protected `main` is +> `cb187cadee5fb6c46d8a944815ccc154a1e028d1` after unauthorized squash +> of leftover-map coordinates (v2.24.0 / #782). Revert #808 +> (`revert-pr782-unauthorized` @ `1af3e53e`) restores `main` toward +> `fc13acaa` (v2.23.0) and still needs independent APPROVE. GitHub +> writes work (comment/close/create-PR/push). Token still has empty +> `X-OAuth-Scopes`; repo permission is ADMIN. Open leftover stack +> still lacks independent APPROVE. Do not self-approve. +> #782 leftover-map coordinates through pair-segment distance +> (v2.24.0–v2.28.0 / ADR 0267–0271) is `9bdd3e4d`. #802 explained +> leftover share on pair segments (v2.30.0 / ADR 0273, includes v2.29 +> reconstruction) is `79ec22f0`. #803 unexplained leftover share on pair +> segments (v2.31.0 / ADR 0274) is `089a1571`. #804 leftover-map cross +> share on pair segments (v2.32.0 / ADR 0275) is `c513002b`. #805 leftover-map +> unexplained leftover `U` on pair segments (v2.33.0 / ADR 0276) is +> `e3fdfd74`. #806 leftover residual `R` on pair segments (v2.34.0 / +> ADR 0277) is `33f6c3dd`. #809 leftover observed `Y` on pair segments +> (v2.35.0 / ADR 0278) is `b334b00e`. #810 leftover expected `E` on pair +> segments (v2.36.0 / ADR 0279) is `6e37757a`. #811 leftover-map rank on +> pair segments (v2.37.0 / ADR 0280) is `e626a1d0`. #812 leftover-map +> complete-case coverage on the graphic (v2.38.0 / ADR 0281) is `64964cb6`. +> #813 leftover-map item complete-case coverage on the graphic (v2.39.0 / +> ADR 0282) is `de2a8a8b`. Leave +> #782/#802/#803/#804/#805/#806/#808/#809/#810/#811/#812/#813 open for +> independent review. Do not squash-merge stacked leftover PRs onto an +> unprotected leftover base. Do not merge #808 without independent APPROVE. +> Strix on #782 failed closed at `Run Strix (quick)`; do not weaken +> fail-closed on `Vulnerabilities [1-9]`. Org Strix `ModelBehaviorError` +> classifier is already on ContextualWisdomLab/.github. Copilot review is +> not independent APPROVE. Issues #79 and #87 stay open. #96 is already +> closed. Only collaborator is `seonghobae`; no independent reviewer can +> be requested from this token. +> +> Next buyer increment on this cycle: leftover-map incomplete post +> coverage on the graphic display (ADR 0283 / v2.40.0) delivered locally on +> `feat/leftover-map-plot-incomplete-v2400`. Caption the leftover-map +> graphic with persisted leftover-map incomplete post coverage so a +> `used N of M scored posts` caption is not read as every scored post +> entering the map. UI-only; no new columns. Missing, non-integer, or +> negative dropped count, or a dropped count that contradicts usable +> complete-case integers, omits that leftover-map incomplete post caption. +> Dropped `0` is shown when persisted. Do not invent dropped posts from +> scored minus used, plotted marker count, leftover-map distance, +> leftover-map rank, leftover-map post coverage, leftover-map item +> coverage, or the count of unused axes. Do not invent leftover scores. +> Stack onto leftover branch `feat/leftover-map-plot-item-coverage-v2390` +> / #813; leave the PR open for independent review. Do not squash-merge +> onto the leftover base. Do not persist leftover-map inner product, +> cosine, or length as separate columns. Incomplete item coverage on the +> graphic remains unnamed. + > Exact-head loop overlay: 2026-08-30 19:48 KST. Protected `main` is > `cb187cadee5fb6c46d8a944815ccc154a1e028d1` after unauthorized squash > of leftover-map coordinates (v2.24.0 / #782). Revert #808 diff --git a/frontend/package.json b/frontend/package.json index ae62f2d9c..7c43d0863 100644 --- a/frontend/package.json +++ b/frontend/package.json @@ -1,7 +1,7 @@ { "name": "frontend", "private": true, - "version": "2.39.0", + "version": "2.40.0", "type": "module", "scripts": { "dev": "vite", diff --git a/frontend/src/App.test.tsx b/frontend/src/App.test.tsx index fb3afaf0c..8af153295 100644 --- a/frontend/src/App.test.tsx +++ b/frontend/src/App.test.tsx @@ -4160,6 +4160,9 @@ describe("App, authenticated", () => { expect(screen.getByLabelText("Leftover-map graphic item coverage")).toHaveTextContent( "Leftover map used 2 of 2 scored criteria (complete-case)", ); + expect(screen.getByLabelText("Leftover-map graphic incomplete posts")).toHaveTextContent( + "Leftover map dropped 1 incomplete posts", + ); const coverageCaption = screen.getByLabelText("Leftover map coverage"); const closestPair = screen.getByRole("button", { name: /open leftover closest pair: public post/i }); const farthestPair = screen.getByRole("button", { diff --git a/frontend/src/components/LeftoverMapPlot.css b/frontend/src/components/LeftoverMapPlot.css index 177e46ada..7845798f1 100644 --- a/frontend/src/components/LeftoverMapPlot.css +++ b/frontend/src/components/LeftoverMapPlot.css @@ -8,7 +8,8 @@ .leftover-map-plot-caption, .leftover-map-plot-coverage, -.leftover-map-plot-item-coverage { +.leftover-map-plot-item-coverage, +.leftover-map-plot-incomplete-posts { margin: 0 0 var(--space-control-gap); color: var(--color-text); font-size: 0.85rem; diff --git a/frontend/src/components/LeftoverMapPlot.test.tsx b/frontend/src/components/LeftoverMapPlot.test.tsx index 98c7700f9..aafe4a9fa 100644 --- a/frontend/src/components/LeftoverMapPlot.test.tsx +++ b/frontend/src/components/LeftoverMapPlot.test.tsx @@ -61,7 +61,7 @@ describe("LeftoverMapPlot", () => { expect(screen.getByLabelText("Leftover-map graphic display")).toBeInTheDocument(); expect( screen.getByText( - "Leftover map after IRT main effects. Axis ticks name persisted leftover-map coordinates. Pair segments name leftover-map distance d, leftover-map reconstruction R̂, leftover-map explained leftover share e, leftover-map unexplained leftover share s, leftover-map cross share x, leftover-map unexplained leftover U, leftover residual R, leftover observed Y, leftover expected E, and leftover-map rank. The plot names leftover-map complete-case coverage and leftover-map item complete-case coverage when persisted. Click a post marker to open that post. The plot does not invent a leftover score.", + "Leftover map after IRT main effects. Axis ticks name persisted leftover-map coordinates. Pair segments name leftover-map distance d, leftover-map reconstruction R̂, leftover-map explained leftover share e, leftover-map unexplained leftover share s, leftover-map cross share x, leftover-map unexplained leftover U, leftover residual R, leftover observed Y, leftover expected E, and leftover-map rank. The plot names leftover-map complete-case coverage, leftover-map item complete-case coverage, and leftover-map incomplete post coverage when persisted. Click a post marker to open that post. The plot does not invent a leftover score.", ), ).toBeInTheDocument(); expect(screen.getByText("Post ξ")).toBeInTheDocument(); @@ -91,6 +91,7 @@ describe("LeftoverMapPlot", () => { expect(screen.getAllByLabelText("leftover-map rank rank 1")).toHaveLength(2); expect(screen.queryByLabelText("Leftover-map graphic coverage")).not.toBeInTheDocument(); expect(screen.queryByLabelText("Leftover-map graphic item coverage")).not.toBeInTheDocument(); + expect(screen.queryByLabelText("Leftover-map graphic incomplete posts")).not.toBeInTheDocument(); const postMarker = screen.getByRole("button", { @@ -620,6 +621,9 @@ describe("LeftoverMapPlot", () => { expect(screen.getByLabelText("Leftover-map graphic item coverage")).not.toHaveTextContent( "Leftover map used 2 of 2 scored criteria (complete-case)", ); + expect(screen.getByLabelText("Leftover-map graphic incomplete posts")).toHaveTextContent( + "Leftover map dropped 1 incomplete posts", + ); }); it("omits leftover-map item coverage on the graphic when item coverage is missing or not usable", () => { @@ -646,6 +650,9 @@ describe("LeftoverMapPlot", () => { expect(screen.getByLabelText("Leftover-map graphic coverage")).toHaveTextContent( "Leftover map used 2 of 3 scored posts (complete-case)", ); + expect(screen.getByLabelText("Leftover-map graphic incomplete posts")).toHaveTextContent( + "Leftover map dropped 1 incomplete posts", + ); }); it("names leftover-map item coverage 0 of M on a rank-0 origin when that persisted used count is a non-negative integer", () => { @@ -691,5 +698,58 @@ describe("LeftoverMapPlot", () => { expect(screen.getByLabelText("Leftover-map graphic coverage")).toHaveTextContent( "Leftover map used 0 of 3 scored posts (complete-case)", ); + expect(screen.getByLabelText("Leftover-map graphic incomplete posts")).toHaveTextContent( + "Leftover map dropped 3 incomplete posts", + ); + }); + + it("omits leftover-map incomplete posts on the graphic when the dropped count is missing or not usable", () => { + const { rerender } = render( + , + ); + expect(screen.queryByLabelText("Leftover-map graphic incomplete posts")).not.toBeInTheDocument(); + rerender( + , + ); + expect(screen.queryByLabelText("Leftover-map graphic incomplete posts")).not.toBeInTheDocument(); + expect(screen.getByLabelText("Leftover-map graphic coverage")).toHaveTextContent( + "Leftover map used 2 of 3 scored posts (complete-case)", + ); + }); + + it("names leftover-map incomplete posts 0 when that persisted dropped count is a non-negative integer", () => { + render( + , + ); + expect(screen.getByLabelText("Leftover-map graphic incomplete posts")).toHaveTextContent( + "Leftover map dropped 0 incomplete posts", + ); + expect(screen.getByLabelText("Leftover-map graphic coverage")).toHaveTextContent( + "Leftover map used 3 of 3 scored posts (complete-case)", + ); }); }); diff --git a/frontend/src/components/LeftoverMapPlot.tsx b/frontend/src/components/LeftoverMapPlot.tsx index 23bf202c9..7b11b7b4e 100644 --- a/frontend/src/components/LeftoverMapPlot.tsx +++ b/frontend/src/components/LeftoverMapPlot.tsx @@ -3,9 +3,12 @@ import { t, tf } from "../i18n"; import { formatLeftoverMapCoordinatePair } from "../leftoverMapCoordinates"; import { leftoverMapCoverageCounts, + leftoverMapIncompletePostCount, leftoverMapItemCoverageCounts, LEFTOVER_MAP_PLOT_COVERAGE, LEFTOVER_MAP_PLOT_COVERAGE_LABEL, + LEFTOVER_MAP_PLOT_INCOMPLETE_POST, + LEFTOVER_MAP_PLOT_INCOMPLETE_POST_LABEL, LEFTOVER_MAP_PLOT_ITEM_COVERAGE, LEFTOVER_MAP_PLOT_ITEM_COVERAGE_LABEL, } from "../leftoverMapCoverage"; @@ -72,8 +75,9 @@ function leftoverMapPlotAxisText( * unexplained leftover ``U``, leftover residual ``R``, leftover * observed ``Y``, leftover expected ``E``, and leftover-map rank so the * pair-row badges match the graphic. Name leftover-map complete-case - * coverage and leftover-map item complete-case coverage on the figure when - * those persisted post and criterion counts are usable. + * coverage, leftover-map item complete-case coverage, and leftover-map + * incomplete post coverage on the figure when those persisted post and + * criterion counts are usable. * Omit that distance caption when ``d`` is missing or non-finite. Omit * that reconstruction caption when ``R̂`` is missing or non-finite. Omit * that explained leftover share caption when ``e`` is missing or @@ -87,7 +91,8 @@ function leftoverMapPlotAxisText( * caption when rank is missing, negative, or not an integer. Omit that leftover-map * coverage caption when coverage is missing or not usable complete-case integers. * Omit that leftover-map item coverage caption when item coverage is missing or - * not usable complete-case integers. + * not usable complete-case integers. Omit that leftover-map incomplete post + * caption when incomplete post coverage is missing or not a usable integer. * Omit that axis badge when share is * missing or non-finite and keep the existing leftover-map axis text. * Omit the plot when no pair has four finite leftover-map coordinates. @@ -106,6 +111,7 @@ export function LeftoverMapPlot({ } const coverageCounts = leftoverMapCoverageCounts(leftoverMapCoverage); const itemCoverageCounts = leftoverMapItemCoverageCounts(leftoverMapCoverage); + const incompletePostCount = leftoverMapIncompletePostCount(leftoverMapCoverage); const openPost = (postId: string) => { const pair = firstPlottablePairForPost(pairs, postId); @@ -135,6 +141,15 @@ export function LeftoverMapPlot({ {tf(LEFTOVER_MAP_PLOT_ITEM_COVERAGE, itemCoverageCounts)}

) : null} + {incompletePostCount !== null ? ( +

+ {tf(LEFTOVER_MAP_PLOT_INCOMPLETE_POST, incompletePostCount)} +

+ ) : null}