Skip to content

Conversation

@msimberg
Copy link
Collaborator

No description provided.

@github-actions
Copy link

preview available: https://docs.tds.cscs.ch/243

## Running CP2K

### Running on the HPC platform
### Running on GH200 nodes
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We could call this Running on Daint as well? I left the heading further down as Running on Eiger, so it's a bit inconsistent like this. Open to better suggestions.

Motivation to change it: HPC platform includes both daint and eiger. Note to self, change the later mentions of "HPC platform" as well in this section...

Copy link
Member

Choose a reason for hiding this comment

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

Running on Daint: CP2K is deployed on the HPC platform only, and users on that platform might be familiar with Daint, but not ght200.

Copy link
Member

Choose a reason for hiding this comment

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

Maybe the first sentence could say something like: Daint nodes have 4 grace-hopper GPUs that have to be configured properly to get the best performance.?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I agree, Running on Daint is both the most consistent option, and one that non-experts will understand.

As @bcumming suggests, it is probably a good idea to briefly remind users of the architecture of Daint (which was kinda self-contained in the GH200 title, but for expert users only)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks both for having a look. I've updated this to "Running on daint", and a few other places to match hopefully.

@msimberg msimberg marked this pull request as draft August 25, 2025 13:04
@github-actions
Copy link

preview available: https://docs.tds.cscs.ch/243

@msimberg msimberg marked this pull request as ready for review August 25, 2025 15:25
@bcumming bcumming added this pull request to the merge queue Aug 27, 2025
Merged via the queue into eth-cscs:main with commit f687e4f Aug 27, 2025
3 checks passed
@msimberg msimberg deleted the minor-fixes-2 branch August 27, 2025 07:15
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.

3 participants