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

PyTorch 2.0 compatibility: Explicit PyTorch 1.x check causing issues with packages that depend on PyTorch / pyro (e.g. BoTorch) #3162

Closed
danielrjiang opened this issue Dec 9, 2022 · 5 comments · Fixed by #3164

Comments

@danielrjiang
Copy link

Issue Description

The explicit check for PyTorch 1.x here (https://github.com/pyro-ppl/pyro/blob/dev/pyro/distributions/torch_patch.py#L10) is causing problems when another package has a dependency on PyTorch + Pyro, since PyTorch is now at 2.0. For example, it is causing BoTorch tests to fail here (pytorch/botorch#1551).

Could this check be removed to allow for PyTorch 2.0?

Environment

Mac OS 11.7.1
Python 3.10
PyTorch 2.0
Pyro 1.8.3

Code Snippet

https://github.com/pytorch/botorch/actions/runs/3659534850/jobs/6185642011

@fritzo fritzo mentioned this issue Dec 12, 2022
@fritzo
Copy link
Member

fritzo commented Dec 12, 2022

Sure, thanks for pointing out this remnant from torch 0.x days.

Do y'all want a pyro-ppl release, or is it enough to have #3164 in Pyro's dev branch?

@Balandat
Copy link
Contributor

@danielrjiang how do you feel about testing against pyro dev (GH install) with torch nightlies? If that works we're probably ok without a release for now.

Thanks!

@Balandat
Copy link
Contributor

Balandat commented Jan 3, 2023

@fritzo are there any plans to put out a new pyro-ppl soon? It would be nice to not have to work off the dev branch when doing (integration) testing on pytorch. Also the pytorch 2.0 release is coming up in March and you probably want to make sure not everything breaks for everyone using pyro when that release happens :)

@fritzo
Copy link
Member

fritzo commented Jan 3, 2023

@Balandat sure, we can do a minor release of pyro-ppl

@fritzo
Copy link
Member

fritzo commented Jan 3, 2023

done

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 a pull request may close this issue.

3 participants