-
Notifications
You must be signed in to change notification settings - Fork 49
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
Add Gitlab CI/CD targets for gh200 #1546
Conversation
cscs-ci run default |
This reverts commit 77aa1b9.
cscs-ci run default |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know how to solve the version constraints problem, but we will discuss today if we can drop 3.8.
cscs-ci run default |
ipython==8.12.3 # via -c constraints.txt, ipykernel | ||
isort==5.13.2 # via -c constraints.txt, -r requirements-dev.in | ||
jax[cpu]==0.4.13 # via -c constraints.txt, gt4py (pyproject.toml) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are removing jax from requirements-dev.txt because it's build for python 3.8. For me that's fine as we anyway don't have jax developers atm. Once we drop 3.8 and 3.9 we will have it back. What do you think @egparedes ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed with Enrique. Looks good.
Co-authored-by: Hannes Vogt <hannes@havogt.de>
cscs-ci run default |
This PR adds the CI/CD configuration for GH200 nodes, while keeping the currrent configuration for x86_64+CUDA on PizDaint.
Some differences between GH200 (on Todi vCluster) and x86_64+CUDA (on PizDaint):
char
is signed or unsigned. The Python bindings in GT4Py cartesian rely on the signed representation, which was the default for the compiler on the x86_64 Ubuntu base image. This behavior is not the default on ARM base image, so we have to enforce it with the flag-fsigned-char