Skip to content

feat(psychometric): restore Driver p.16 discreteDIFFUSIONstd Q_Δt/p on main - #311

Draft
seonghobae wants to merge 1 commit into
mainfrom
agent/psychometric-discrete-diffusion-std-20260831
Draft

feat(psychometric): restore Driver p.16 discreteDIFFUSIONstd Q_Δt/p on main#311
seonghobae wants to merge 1 commit into
mainfrom
agent/psychometric-discrete-diffusion-std-20260831

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Restores Driver, Oud, and Voelkle (2017, p. 16) scalar discreteDIFFUSIONstd Q_Δt / p onto current main after 0ce16e8 dropped the pre-consolidation code while research notes already named the map (register items 57–58).

Ports the #296 slice onto 6444a81. Does not cherry-pick the conflicting #296 commits.

Form strictly positive asymDIFFUSION p = −q / (2 a) first, then Q_Δt from Equation 4, then the ratio. Scalar stationary identity is 1 − exp(2 a Δt). The 2017-era summary.ctsemFit.R forms unstandardised discreteDIFFUSION when verbose=TRUE and does not form a discreteDIFFUSIONstd matrix. Printed p. 16 example is discreteDRIFTstd. Footnote 4 uses only the relevant within-subject variance.

Fail-closed: zero q, a ≥ 0, non-event clock, non-positive Δt.
Refuse: unstandardised Q_Δt, continuous q/p = −2 a (DIFFUSIONstd), trait-contaminated Q_Δt / (trait + p + added).

Supersedes conflicting #296. Does not invent DIFFUSIONstd (register 59). Does not convert #310.

Local cargo test -p psychometric_core is green. Meredith (1993) and Mislevy (1991) remain unread (Unpaywall is_oa: false). Still not a Kalman filter, not a matrix expm, not ESEM estimation, not DSEM, and not ctsem estimation.

Head is c73b250. Do not merge without independent APPROVE and green product CI. Stays Draft.

…n main

Port the scalar Q_Δt / p map onto current main after 0ce16e8 dropped the
pre-consolidation code. Form strictly positive asymDIFFUSION first.
Refuse unstandardised Q_Δt, continuous q/p = −2a, and trait-contaminated
Q_Δt/(trait+p+added). 2017-era summary.ctsemFit.R does not form a
discreteDIFFUSIONstd matrix. Not Kalman, not matrix expm, not DSEM.
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

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.

@seonghobae seonghobae left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Requesting Copilot review of discreteDIFFUSIONstd Q_Δt/p restore.

Copy link
Copy Markdown
Contributor Author

@github-copilot review this exact head c73b250. Requesting Copilot code review of the discreteDIFFUSIONstd restore.

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

Labels

enhancement New feature or request priority: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant