Skip to content

Introduce a minimal potential-response core - #41

Merged
rajarshi-mukherjee24 merged 4 commits into
stat-lib:mainfrom
bocowgill:codex/potential-response-core
Sep 17, 2026
Merged

rajarshi-mukherjee24 merged 4 commits into
stat-lib:mainfrom
bocowgill:codex/potential-response-core

Conversation

@bocowgill

@bocowgill bocowgill commented Aug 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Define a generic, assumption-free PotentialResponse type.
  • Add pointwise selection and same-unit substitution.
  • Add application, identity, and associativity laws.

Scope

This PR is purely functional. It introduces no probability or measurability structure, causal assumptions, estimands, identification results, estimators, statistical inference, or dependency on structural or graphical APIs.

Verification

  • lake build
  • lake lint
  • lake exe mk_all --check

Closes #40.

@bocowgill
bocowgill marked this pull request as ready for review August 28, 2026 22:39
@zixiaowang17

Copy link
Copy Markdown
Collaborator

Hi @bocowgill I have approved the building

@bocowgill

Copy link
Copy Markdown
Contributor Author

Thanks—the build passed!

@bocowgill

Copy link
Copy Markdown
Contributor Author

@rajarshi-mukherjee24: this is ready for review and CI passes. Could you take a look, or suggest an appropriate reviewer? It implements the revised scope of #40, including the documentation clarifications and substitution laws we discussed. Thanks!

@bocowgill

Copy link
Copy Markdown
Contributor Author

@zixiaowang17 Could you approve the updated build when convenient? This push only adds the nested-response documentation example requested on #40. Thanks!

@bocowgill

Copy link
Copy Markdown
Contributor Author

@rajarshi-mukherjee24 The identity-response change and the documentation example are now complete, the issue description matches the implementation, and CI passes. This is ready for final review and merge.

@rajarshi-mukherjee24
rajarshi-mukherjee24 merged commit ba20712 into stat-lib:main Sep 17, 2026
1 check passed
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.

[Question] Introduce a minimal potential-response core

3 participants