Skip to content

ICON-DSL uenv: ICON-Exclaim with ICON4Py dycore. #7

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

DropD
Copy link
Collaborator

@DropD DropD commented Jun 10, 2025

No description provided.

@DropD DropD requested a review from bcumming June 10, 2025 12:12
gh200: '25.8/gh200'
deploy:
santis: [gh200]
develop: False
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not sure about this: The UENV is still in development, does that mean this should be True?

Copy link
Contributor

Choose a reason for hiding this comment

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

No. This refers to whether you are using a recent/develop branch of stackinator.
Set it to False.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

👍

- icon4py@icon_20250328 +distributed +cuda
- py-mpi4py
- python
unify: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Think about adding variants, so that any cuda code is compiled correctly.

Also, I notice that you are installing rust and rust bootstrap, in order to install uv. This is a lot of overhead for something that can be downloaded as a stand alone binary.

@bcumming
Copy link
Contributor

cscs-ci run alps;system=santis;uarch=gh200;uenv=icon-dsl:25.8

@bcumming
Copy link
Contributor

cscs-ci run alps;system=santis;uarch=gh200;uenv=icon-dsl:25.8

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