Skip to content

docs(adr): define Rust polytomous period artifact - #1436

Draft
seonghobae wants to merge 16 commits into
feat/interaction-map-rust-summary-1412from
feat/polytomous-period-artifact-adr
Draft

seonghobae wants to merge 16 commits into
feat/interaction-map-rust-summary-1412from
feat/polytomous-period-artifact-adr

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Outcome

Defines the owner-side Rust artifact required to remove active Python GRM/GPCM period-report arithmetic from downstream consumers without preserving an unsupported weighted model-selection score.

The proposed contract keeps authorization/period state downstream, moves matrix construction and measurement summaries into one Rust-owned artifact, requires formal distinguishability before GRM/GPCM preference, returns explicit unavailable/indeterminate states instead of a Python numerical fallback, and requires recovery/parity/fingerprint evidence before acceptance.

Stack / freshness — 2026-09-03

This Draft is intentionally stacked on #1417 because it reuses the residual interaction-map envelope.

  • exact parent feat(interaction-map): extend Rust result envelope #1417: feat/interaction-map-rust-summary-1412@3cdc684a94d7cdc0c30c0db773c0e92cfd9492f2;
  • predecessor child: 78a54ed032635713d57a803077feb4e92e71a9d1;
  • exact current child: feat/polytomous-period-artifact-adr@d7387d7e09e1c7cd45d606ece96dd054d06cb843.

Fresh comparison before repair showed the predecessor child diverged from the current #1417 parent (behind_by=2) while its only effective child-owned delta remained docs/adr/0027-polytomous-period-artifact.md. The child was repaired non-destructively: d7387d7e... records predecessor 78a54ed... as first parent and exact current #1417 parent 3cdc684... as second parent, with a tree constructed from the exact parent tree plus the already-reviewed ADR blob. No force push, destructive rebase, unrelated source copy, or numerical change was used.

#1417 must land first. Any future parent movement invalidates child exact-head/base evidence and requires fresh comparison before another stack action.

Verification

Every predecessor child workflow/review result is historical after d7387d7e.... Keep this PR Draft until the unchanged exact child head has regenerated applicable hosted CI/security/documentation evidence and the live review/protection policy is satisfied.

No numerical implementation, Python numerical fallback, heuristic model-selection score, or psychometric/statistical arithmetic is added.

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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

Open in Devin Review

Comment thread docs/adr/0022-polytomous-period-artifact.md
@seonghobae
seonghobae marked this pull request as draft August 26, 2026 08:13
@seonghobae seonghobae added documentation Improvements or additions to documentation priority: medium Normal-priority or P2 work status: draft Draft pull request type: docs Documentation, ADR, PRD, or technical writing labels Sep 2, 2026 — with ChatGPT Codex Connector

This branch has not been deployed

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

Labels

documentation Improvements or additions to documentation priority: medium Normal-priority or P2 work status: draft Draft pull request type: docs Documentation, ADR, PRD, or technical writing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant