-
Notifications
You must be signed in to change notification settings - Fork 486
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
Transition to Pytorch 2.0 #1229
Comments
Should NP be transitioning to Pytorch Lightning 2.0? https://lightning.ai/pages/blog/training-compiled-pytorch-2.0-with-pytorch-lightning |
It is a bit tricky to get a good overview as one issue leads to another. |
There is a migration guide for lightning: https://lightning.ai/docs/pytorch/latest/upgrade/from_1_9.html |
Using
This can be circumvented by installing the cpu only version
As far as I know, there is no current This however is not an issue for pytorch versions before 2.0. While it seems like the GPU version is installed, this will still run fine on machines without CUDA. |
I've done some further testing, but unfortunately it does not seem possible to reliably install the CPU version of pytorch using poetry for versions 2.0.0 and above. However, if we're willing to use pip on top of poetry, installing the CPU version of torch is pretty straightforward with the command However, this makes the installation process more complicated than it otherwise would have been with torch v1. Feedback and ideas would be welcome! |
@hxyue1 thanks for flagging this. I was a bit confused already why you opted for the You are always free to choose which package manager you use, so if pip works, that's great. I would not force the I am on macOS with an Intel chip with no accelerator, so it works for me. How bad is it on other systems? I assume the problem only arises if you use |
@noxan as far as I'm aware, the CI tests use poetry which by default will install the GPU version on Ubuntu. This unfortunately causes the tests to fail because the Github runners are CPU only (as far as I know). Which is why as a workaround I attempted to force a CPU only install for the CI tests. Would it be possible to change the github actions to use pip instead of Poetry? |
@hxyue1 Ah I see you were talking about CI. On my pull request the test do not fail on Ubuntu 🤔 |
Great, that's fantastic! I have no idea what I'm doing wrong then... |
done in #1404 |
No description provided.
The text was updated successfully, but these errors were encountered: