8000 Add CSCS CI by havogt · Pull Request #1818 · GridTools/gridtools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add CSCS CI #1818

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 64 commits into from
Feb 26, 2025
Merged

Add CSCS CI #1818

merged 64 commits into from
Feb 26, 2025

Conversation

havogt
Copy link
Contributor
@havogt havogt commented Feb 19, 2025

Add a minimal CI for CSCS Alps GH200.

@havogt
Copy link
Contributor Author
havogt commented Feb 19, 2025

cscs-ci run

4 similar comments
@havogt
Copy link
Contributor Author
havogt commented Feb 19, 2025

cscs-ci run

@havogt
Copy link
Contributor Author
havogt commented Feb 19, 2025

cscs-ci run

@havogt
Copy link
Contributor Author
havogt commented Feb 19, 2025

cscs-ci run

@havogt
Copy link
Contributor Author
havogt commented Feb 19, 2025

cscs-ci run

@havogt
Copy link
Contributor Author
havogt commented Feb 19, 2025

cscs-ci run

@havogt
Copy link
Contributor Author
havogt commented Feb 19, 2025

cscs-ci run

1 similar comment
@havogt
Copy link
Contributor Author
havogt commented Feb 19, 2025

cscs-ci run

@havogt
Copy link
Contributor Author
havogt commented Feb 19, 2025

cscs-ci run

@havogt
Copy link
Contributor Author
havogt commented Feb 25, 2025

cscs-ci run

1 similar comment
@havogt
Copy link
Contributor Author
havogt commented Feb 25, 2025

cscs-ci run

@@ -32,6 +32,7 @@ namespace gridtools::fn {
return neighbor_table::neighbors(table, index);
}

#if defined(__NVCC__) && defined(__CUDACC_VER_MAJOR__) && (__CUDACC_VER_MAJOR__ < 12 || (__CUDACC_VER_MAJOR__ == 12 && __CUDACC_VER_MINOR__ > 8))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

see #1822

Copy link
Contributor
@msimberg msimberg left a comment

Choose a reason for hiding this comment

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

Looks fine to me.

Comment on lines +55 to +56
echo be0d91732d5b0cc6fbb275c7939974457e79b54d6f07ce2e3dfdd68bef883b0b boost_1_85_0.tar.gz > boost_hash.txt && \
sha256sum -c boost_hash.txt && \
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor: just as a shortcut you can also do this:

Suggested change
echo be0d91732d5b0cc6fbb275c7939974457e79b54d6f07ce2e3dfdd68bef883b0b boost_1_85_0.tar.gz > boost_hash.txt && \
sha256sum -c boost_hash.txt && \
(echo be0d91732d5b0cc6fbb275c7939974457e79b54d6f07ce2e3dfdd68bef883b0b boost_1_85_0.tar.gz | sha256sum --check) && \

? (Also minor: I much prefer the long versions of options like --check since there's less guessing what c happens to stand for in this particular case).

Then again, I guess this will be removed soon enough anyway...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll not touch it, because I want to delete it later today.

@havogt havogt merged commit 0299781 into master Feb 26, 2025
69 checks passed
@havogt havogt deleted the cscsci branch February 26, 2025 11:58
havogt added a commit that referenced this pull request Apr 11, 2025
Add a minimal CI for CSCS Alps GH200.
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
0