Skip to content

chore(stack): place crossed estimator after longitudinal foundation - #1016

Closed
seonghobae wants to merge 16 commits into
feat/multiple-membership-current-review-985from
fix/longitudinal-current-review-976
Closed

chore(stack): place crossed estimator after longitudinal foundation#1016
seonghobae wants to merge 16 commits into
feat/multiple-membership-current-review-985from
fix/longitudinal-current-review-976

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Temporary branch-reconciliation PR. Merge parent #1005 exact head ed9868c1b3636d0470dc82367e80a43036483018 into child #1014 exact branch head 34697b6df58c1424654bc890571a3dfbe806fd97.

The branches share multilevel exports/bindings and canonical documentation, but their numerical kernels are distinct: #1005 owns Rust longitudinal state/CT-AR Rasch likelihood, optimization and uncertainty; #1014 owns Rust crossed/weighted multiple-membership u_h MAP/Newton estimation and recovery. Preserve both sets. After this normal non-force merge, retarget #1014 to #1005's branch so its effective diff contains only the crossed-estimator slice. Do not merge this temporary PR to main as a standalone landing unit.

cursoragent and others added 16 commits August 17, 2026 16:37
Land a focused successor to #848 on live main: independent per-respondent
OLS trends and caller-supplied discrete AR predictions, with honest
estimand metadata, fail-closed worker joins, checked AR gaps, and
scale-relative slope degeneracy. Number the decision ADR-0018 so it does
not collide with main ADR-0015 or #948 ADRs 0016/0017.

Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
Use two-token snake_case respondent IDs and cover NumPy scalar
observation conversion on the public longitudinal boundary.

Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
PR 948 already records Angoff delta-plot and Bradley-Terry MM. Carry
those accepted ADRs in this branch so the index does not skip numbers
and a later merge with the citation work does not drop them.

Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
Close the leftover llvm-cov paths in the longitudinal state engine: skip an
unused worker chunk, test first-to-last sequence-span underflow as a helper,
and recover an AR series that starts after a leading missing occasion.
Accept Python int and NumPy integer scalars at the public boundary.

Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
The required default-setup Analyze (actions) job is still the 17:40 UTC
init failure (feature-enablement HTTP 503). The repository CodeQL copy
already passed on a later rerun. This integration cannot call
`gh run rerun` (403, needs actions: write), so retrigger the stale org
check without changing product code.

Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
* Add joint MAP hierarchical continuous-time AR(1) Rasch slice.

Introduce a Rust-owned jointly estimated longitudinal IRT kernel stacked
on the #976 OLS/AR state layer: shared (mu, tau, lambda), elapsed-day
OU transitions, measurement-information Wald state intervals, and
honest estimand metadata. Multiple-membership u_h and GPU parity are
excluded. Python remains marshalling-only.

Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>

* Accept NumPy intercept vectors and hide test-only scale helpers.

NumPy 2 no longer treats ndarray as a Sequence, so the hierarchical
simulator now accepts both sequences and arrays. Empirical-scale helpers
used only by unit tests are cfg(test) so the production lib stays clean.

Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Seongho Bae <seonghobae@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: c654413b-a7d9-49f7-9ab3-2a0da2a63426

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

Closing this temporary conflict-visualization PR unmerged. It is not a product landing vehicle and is not safe to merge to either branch. The actual same-branch reconciliation request remains on #1014: create one non-force two-parent merge commit on feat/multiple-membership-current-review-985, preserve both scientific slices across the nine shared files, then retarget #1014 to #1005's branch. Closing this PR does not close or abandon either product lane.

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.

2 participants