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

Error with Pyro version 1.9.0 #538

Open
devonjkohler opened this issue Apr 24, 2024 · 1 comment · Fixed by #518
Open

Error with Pyro version 1.9.0 #538

devonjkohler opened this issue Apr 24, 2024 · 1 comment · Fixed by #518
Labels
bug Something isn't working upstream

Comments

@devonjkohler
Copy link

devonjkohler commented Apr 24, 2024

Hello,

I've noticed an error when running the package with the newest version of Pyro.

Importing chirho.interventional.handlers.do returns the following error:

Traceback (most recent call last):
File "", line 1, in
File "/root/miniconda3/envs/py311/lib/python3.11/site-packages/chirho/interventional/init.py", line 1, in from . import handlers # noqa: F401
^^^^^^^^^^^^^^^^^^^^^^
File "/root/miniconda3/envs/py311/lib/python3.11/site-packages/chirho/interventional/handlers.py", line 121, in
do = pyro.poutine.handlers._make_handler(Interventions)[1]
TypeError: 'function' object is not subscriptable

Rolling this back to Pyro 1.8 fixes this issue.

@SamWitty
Copy link
Collaborator

Hello @devonjkohler. Thanks for your interest in ChiRho, and for pointing out the compatibility issue.

This is a known issue, and will be addressed very soon when we cut a release including #518 . In fact, #537 brought up the same issue. We've pinned the pyro version in the setup.py file here: https://github.com/BasisResearch/chirho/blob/master/setup.py#L44.

I'll leave this issue up until we cut the next release, as others are likely to encounter the same compatibility issue.

@SamWitty SamWitty linked a pull request Apr 25, 2024 that will close this issue
@eb8680 eb8680 added bug Something isn't working upstream labels Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working upstream
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants