Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Query handlers should be decoupled from Pyro aliases #388

Open
2 of 5 tasks
eb8680 opened this issue Nov 27, 2023 · 0 comments
Open
2 of 5 tasks

Query handlers should be decoupled from Pyro aliases #388

eb8680 opened this issue Nov 27, 2023 · 0 comments

Comments

@eb8680
Copy link
Contributor

eb8680 commented Nov 27, 2023

We currently use chirho.interventional.handlers.do and chirho.observational.handlers.condition in most user-facing code and documentation because they are drop-in replacements for pyro.do and pyro.condition respectively.

However, using do and condition in library code leads to some internal inconsistencies in API design and semantics that will be annoying to work around in the future. This issue tracks work on switching to using more idiomatic and consistently specified query handlers internally.

Tasks:

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

No branches or pull requests

1 participant