Skip to content
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 Milan Support to remap_restarts #40

Closed
mathomp4 opened this issue Nov 30, 2023 · 3 comments
Closed

Add Milan Support to remap_restarts #40

mathomp4 opened this issue Nov 30, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@mathomp4
Copy link
Member

At the moment, remap_restarts doesn't know about SLES15 or the Milan nodes. This is doable but we'll have to rely on CMake to alter the code like we do in gcm_setup (since we know at build time where something was built, we can then change the "allowed" constraints).

I can work with @weiyuan-jiang on this.

@mathomp4 mathomp4 added the enhancement New feature or request label Nov 30, 2023
@atrayano
Copy link

Most of the issues there are related to the SBATCH syntax. The important changes are the 'qos' is not supported, and --ntaska has to be replaced with --nodes and --ntasks-per-node. Also I had to do --partition=scutest

I already made changes and committed them on a brach: bugfix/atrayano/fixes_for_milan

This branch is not meant to be a PR. These are changes I made to get quicky my regridded restarts. The changes need to be done a lot more carefully, to distinguish discover vs discover-mil, and to make sure the scripts work on both systems

@weiyuan-jiang
Copy link
Contributor

Thanks @atrayano

@gmao-rreichle
Copy link
Contributor

addressed in #43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants