Skip to content

Update the CI versions #75

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

Merged
merged 10 commits into from
Dec 16, 2022
Merged

Update the CI versions #75

merged 10 commits into from
Dec 16, 2022

Conversation

raimis
Copy link
Contributor

@raimis raimis commented Dec 13, 2022

  • Include PyTorch 1.12
  • Update from the source installation

@raimis raimis self-assigned this Dec 13, 2022
@raimis raimis marked this pull request as ready for review December 15, 2022 13:48
@raimis raimis marked this pull request as draft December 15, 2022 13:48
@raimis raimis marked this pull request as ready for review December 15, 2022 17:54
@raimis raimis requested a review from peastman December 15, 2022 17:54
@raimis
Copy link
Contributor Author

raimis commented Dec 15, 2022

@peastman could you review?

Comment on lines 43 to 44
# Latest supported versions (with CUDA)
- name: Linux (CUDA 11.2, Python 3.10, PyTorch 1.12)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of these are older versions. How about CUDA 11.8, Python 3.11, and PyTorch 1.13?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python 3.11 and PyTorch 1.13 are not available because there are no corresponding TorchANI packages.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed to CUDA 11.7. CUDA 11.8 isn't available because of the cuda-toolkit action not yet supporting it.

Copy link
Member

@peastman peastman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@raimis raimis merged commit 7491583 into openmm:master Dec 16, 2022
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.

2 participants