Skip to content

feat(reports): leftover-map item complete-case coverage on graphic display (v2.39.0) - #813

Open
seonghobae wants to merge 1 commit into
feat/leftover-map-plot-coverage-v2380from
feat/leftover-map-plot-item-coverage-v2390
Open

feat(reports): leftover-map item complete-case coverage on graphic display (v2.39.0)#813
seonghobae wants to merge 1 commit into
feat/leftover-map-plot-coverage-v2380from
feat/leftover-map-plot-item-coverage-v2390

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • 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 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 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, and a plot caption that matches leftover-map item complete-case coverage; click a post marker or a pair button opens that post.
  • A missing, non-integer, negative-used, non-positive-scored, or used-greater-than-scored item coverage omits that leftover-map item coverage 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, or leftover-map post coverage caption. Coverage 0 of M is shown when that persisted used count is a non-negative integer and scored is a positive integer. Do not invent item coverage from plotted criterion marker count, leftover-map distance, leftover-map rank, leftover-map post coverage, or the count of unused axes.
  • UI-only; no SQL; no new columns. Never invent a leftover score. Never invent a theta. Do not persist leftover-map inner product, cosine, or length. Pair-list note stays post coverage (ADR 0168).
  • Stacked onto leftover-map complete-case coverage on the graphic (feat/leftover-map-plot-coverage-v2380 / feat(reports): leftover-map complete-case coverage on graphic display (v2.38.0) #812 @ 64964cb6). 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

  • frontend vitest: leftoverMapCoverage + leftoverMapPlotLayout + LeftoverMapPlot + LeftoverPairList + i18n + App leftover bits (338 passed)
  • oxlint on changed leftover-map files: 0 warnings, 0 errors
  • tests/test_leftover_pairs.py and 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.


Devin Review

Caption the leftover-map graphic display with persisted leftover-map
item complete-case coverage (ADR 0282). After make seed, closest and
farthest leftover pairs sit above the member list with a plot caption
that matches leftover-map item complete-case coverage; click a post
marker or a pair button opens that post.

A missing, non-integer, negative-used, non-positive-scored, or
used-greater-than-scored item coverage omits that leftover-map item
coverage caption. Coverage 0 of M is shown when that persisted used
count is a non-negative integer. Do not invent item coverage from
plotted criterion marker count, leftover-map distance, leftover-map
rank, leftover-map post coverage, or the count of unused axes. Never
invent a leftover score or a theta. UI-only; no new columns.

Copy link
Copy Markdown
Contributor Author

Stacked onto leftover-map complete-case coverage on the graphic (#812 @ 64964cb). Independent APPROVE required. Do not self-approve. Do not squash-merge onto the leftover base. After independent review, retarget onto leftover-coordinates (#782) rather than squash-merging here.

Copilot/Devin review is not independent APPROVE. Only collaborator is seonghobae; no independent reviewer can be requested from this token.

@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: Pro Plus

Run ID: ea4f98ef-5703-41f2-b2c1-ba45b19f8080

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.

@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: No Issues Found

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

Devin Review

Copy link
Copy Markdown
Contributor Author

Write retry this cycle (2026-08-30 20:12 KST): GitHub comment/close/create-PR/push succeed from this token. Exact head de2a8a8b still has no independent APPROVE (Devin COMMENT is not independent APPROVE). Do not self-approve. Do not squash-merge onto the leftover base. Next buyer increment is leftover-map incomplete post coverage on the graphic (ADR 0283 / v2.40.0), stacked onto this head.

Copy link
Copy Markdown
Contributor Author

Cycle write retry 2026-08-30T20:12+09:00. Head de2a8a8 remains stacked on #812 (feat/leftover-map-plot-coverage-v2380@64964cb6). Do not squash-merge onto an unprotected leftover base. Independent collaborator APPROVE required; Copilot/Devin COMMENT is not independent APPROVE. Next buyer gap ADR 0283 (leftover-map incomplete post coverage on the graphic, v2.40.0) is being stacked locally on this head — not mixed into this PR.

Copy link
Copy Markdown
Contributor Author

Write retry this cycle (2026-08-30 20:26 KST). Exact head de2a8a8b still has no independent APPROVE (Devin COMMENT is not independent APPROVE). Do not self-approve. Do not squash-merge onto the leftover base. Next buyer increment is leftover-map incomplete post coverage on the graphic (ADR 0283 / v2.40.0), stacked onto this head — not mixed into this PR.

Copy link
Copy Markdown
Contributor Author

Stacked #814 (feat/leftover-map-plot-incomplete-v2400 @ 7729287, ADR 0283 / v2.40.0 leftover-map incomplete post coverage on the graphic) onto this head. Leave #813 and #814 open. Do not squash-merge onto this leftover base.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-30T11:26Z write retry: leftover-map item coverage (#813 @ de2a8a8) stays stacked and unmerged. No independent APPROVE exists on this org (only collaborator is seonghobae). Copilot/Devin COMMENT is not independent APPROVE. Do not squash-merge this layer onto leftover-coordinates (#782). Next buyer increment this cycle is leftover-map incomplete post coverage on the graphic (ADR 0283 / v2.40.0) stacked onto this head — UI-only caption from persisted incomplete_post_count / scored_post_count; never invent incomplete as scored−used or from marker count; omit when missing/non-integer/negative/incomplete>scored/scored≤0; show 0 of M. Pair-list note stays post complete-case (ADR 0168). Grouping comparison strip (ADR 0149) does not gain this caption. No new columns. Synthetic fixtures only. NVIDIA_NIM_API_KEY only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant