Skip to content

feat(reports): leftover-map explained leftover share on grouping comparison graphic (v2.68.0) - #851

Open
seonghobae wants to merge 1 commit into
feat/leftover-map-compare-plot-reconstruction-v2670from
feat/leftover-map-compare-plot-explained-share-v2680
Open

feat(reports): leftover-map explained leftover share on grouping comparison graphic (v2.68.0)#851
seonghobae wants to merge 1 commit into
feat/leftover-map-compare-plot-reconstruction-v2670from
feat/leftover-map-compare-plot-explained-share-v2680

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Caption leftover-map explained leftover share on the grouping comparison leftover-map graphic from already-named leftover_map_explained_share (ADR 0311 / v2.68.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, leftover-map complete-case coverage when leftoverMapCoverageCounts returns usable complete-case integers, leftover-map item complete-case coverage when leftoverMapItemCoverageCounts returns usable complete-case integers, leftover-map incomplete post coverage when leftoverMapIncompletePostCount returns a usable dropped integer, leftover-map incomplete item coverage when leftoverMapIncompleteItemCount returns a usable dropped integer, leftover-map reconstruction R̂ when formatLeftoverMapReconstruction returns a usable signed badge, and leftover-map explained leftover share e on that comparison graphic when formatLeftoverMapExplainedShare returns a usable badge; click a post marker or a pair button opens that post.
  • Comparison copy uses accessible name leftover map comparison graphic explained leftover share {label} so it stays distinct from hyphen leftover-map explained leftover share {label} on the period-report graphic and from strip Leftover map comparison explained leftover share. A missing or non-finite e omits that leftover-map comparison graphic explained leftover share caption and keeps leftover-map reconstruction , leftover-map distance d, leftover-map comparison graphic coverage, leftover-map comparison graphic item coverage, leftover-map comparison graphic incomplete posts, and leftover-map comparison graphic incomplete items when those helpers return usable integers. Rank-0 origin cells still name R̂²/R² 0.00 when that persisted explained leftover share is finite.
  • Explained leftover share omits independently of leftover-map reconstruction, leftover-map distance, and coverage notes. A missing explained leftover share omits leftover-map comparison graphic explained leftover share and keeps a usable reconstruction caption and usable coverage notes.
  • This increment does not caption leftover-map unexplained leftover share on the comparison graphic. That unexplained leftover share already sits on the strip through ADR 0295.
  • Do not invent leftover-map explained leftover share from leftover-map reconstruction, leftover residual, leftover-map distance, plotted coordinates, leftover-map unexplained leftover, leftover-map rank, leftover-map axis share, leftover expected, leftover observed, leftover-map post coverage, leftover-map item coverage, leftover-map incomplete post coverage, leftover-map incomplete item coverage, leftover pair count, or the count of unused axes. A finite negative leftover is shown, never clamped.
  • No SQL; no new columns. Distinct from the ADR 0294 comparison strip explained leftover share. Never invent a leftover score. Never invent a theta.
  • Stacked onto leftover-map reconstruction on grouping comparison leftover-map graphic (feat/leftover-map-compare-plot-reconstruction-v2670 / feat(reports): leftover-map reconstruction on grouping comparison graphic (v2.67.0) #850 @ 85451d8). 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

  • pnpm exec vitest run: 799 tests passed.
  • pnpm exec oxlint src: 0 warnings, 0 errors.
  • python3 -m py_compile on lineageweave and backend app modules: ok.
  • PostgreSQL period-report compare tests were not executed here (no live Compose PostgreSQL in this sandbox).

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

…arison graphic (v2.68.0)

Caption persisted leftover-map explained leftover share e on the grouping
comparison leftover-map graphic from already-named leftover_map_explained_share
(ADR 0311). Comparison copy uses leftover map comparison graphic explained
leftover share {label} so it stays distinct from hyphen leftover-map explained
leftover share {label} and from strip Leftover map comparison explained leftover
share. A missing or non-finite e omits that caption and keeps leftover-map
reconstruction and comparison graphic coverage notes. Rank-0 origin cells still
name R̂²/R² 0.00 when that explained leftover share is finite. Do not invent e
from leftover-map reconstruction or leftover residual. Never invent a leftover
score. Never invent a theta. No new columns.
@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: a36bf944-598a-49a8-a7c6-38dfeb9cca9e

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.

Copy link
Copy Markdown
Contributor Author

Queue retry: leftover-map explained leftover share on grouping comparison graphic (v2.68.0 / ADR 0311). Independent APPROVE required before squash-merge. Do not self-approve. Next stacked increment is leftover-map unexplained leftover share on that comparison graphic. Issues #79 and #87 stay open. #96 is already closed.

Copy link
Copy Markdown
Contributor Author

Cycle retry 2026-08-31 06:40 KST: exact head 2d6e8e7 still has no independent APPROVE (CodeRabbit skipped on non-default base). Copilot review requested this cycle. Leave #851 open. Do not self-approve. Do not squash-merge onto the unprotected leftover base. After independent review, retarget onto leftover-coordinates (#782, already on main) rather than squash-merging here.

Next stacked increment (ADR 0312 / v2.69.0) captions leftover-map unexplained leftover share on the grouping comparison leftover-map graphic with distinct accessible name leftover map comparison graphic unexplained leftover share {label}. It does not caption leftover-map unexplained leftover U this increment. Stack onto this PR; leave that PR open for independent review. Issues #79 and #87 stay open.

@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 found 1 potential issue.

Devin Review

Comment thread pyproject.toml
[project]
name = "lineageweave"
version = "2.67.0"
version = "2.68.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Version bump leaves frozen lock stale

The version bump declares 2.68.0, but the frozen lock still identifies lineageweave as 2.65.0. Frozen installs can fail or package the old release.

Prompt for agents
Regenerate uv.lock from the updated pyproject.toml so the local lineageweave package entry records version 2.68.0. Verify the repository's frozen CI sync and backend Docker installation succeed with the refreshed committed lock.
Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31T06:40 KST: retrying GitHub writes (close/merge/comment/push). Head 2d6e8e7 v2.68.0 leftover-map explained leftover share on grouping comparison graphic. Not self-approving. Independent APPROVE required before squash-merge.

Copy link
Copy Markdown
Contributor Author

Stacked next increment opened as #852 (feat/leftover-map-compare-plot-unexplained-share-v2690 @ 2c34896, ADR 0312 / v2.69.0). Leave #851 open for independent review. Exact head 2d6e8e7 still has no independent APPROVE. Do not self-approve. Do not squash-merge.

Copy link
Copy Markdown
Contributor Author

Stacked #852 leftover-map unexplained leftover share on grouping comparison graphic (v2.69.0 / ADR 0312) is open on this head. Independent APPROVE required. Do not self-approve. Do not squash-merge onto the unprotected leftover base.

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