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 ALCF Polaris to E3SM machines #6115

Merged
merged 7 commits into from
Feb 13, 2024
Merged

Add ALCF Polaris to E3SM machines #6115

merged 7 commits into from
Feb 13, 2024

Conversation

amametjanov
Copy link
Member

Add ALCF Polaris to E3SM machines.

[BFB]

@amametjanov amametjanov added Machine Files BFB PR leaves answers BFB labels Dec 9, 2023
@amametjanov amametjanov self-assigned this Dec 9, 2023
Copy link

github-actions bot commented Dec 9, 2023

PR Preview Action v1.4.7
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6115/
on branch gh-pages at 2024-02-06 04:09 UTC

<SAVE_TIMING_DIR>/lus/grand/projects/$PROJECT</SAVE_TIMING_DIR>
<SAVE_TIMING_DIR_PROJECTS>ClimateEnergy_4</SAVE_TIMING_DIR_PROJECTS>
<CIME_OUTPUT_ROOT>/lus/swift/home/$USER/polaris/scratch</CIME_OUTPUT_ROOT>
<DIN_LOC_ROOT>/lus/swift/home/azamatm/polaris/inputdata</DIN_LOC_ROOT>
Copy link
Member

Choose a reason for hiding this comment

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

Is this a permanent location that doesn't depend on an INCITE or ALCC account?

@rljacob
Copy link
Member

rljacob commented Jan 19, 2024

@amametjanov is this ready?

@amametjanov amametjanov marked this pull request as ready for review January 31, 2024 04:15
@amametjanov amametjanov requested a review from rljacob January 31, 2024 04:15
@amametjanov
Copy link
Member Author

This should now be ready for review. I checked with

./cime/scripts/create_test --compiler gnugpu e3sm_gpucxx
./cime/scripts/create_test --compiler nvidiagpu e3sm_gpuacc

<SAVE_TIMING_DIR>/eagle/E3SM_RRM/performance_archive</SAVE_TIMING_DIR>
<SAVE_TIMING_DIR_PROJECTS>SCREAM_Calib,E3SM_RRM</SAVE_TIMING_DIR_PROJECTS>
<CIME_OUTPUT_ROOT>/eagle/E3SM_RRM/$USER/scratch</CIME_OUTPUT_ROOT>
<DIN_LOC_ROOT>/grand/SCREAM_Calib/inputdata</DIN_LOC_ROOT>
Copy link
Member

Choose a reason for hiding this comment

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

Please change this to /grand/E3SMinput The permissions have been fixed.

<CHARGE_ACCOUNT>E3SM_RRM</CHARGE_ACCOUNT>
<SAVE_TIMING_DIR>/eagle/E3SM_RRM/performance_archive</SAVE_TIMING_DIR>
<SAVE_TIMING_DIR_PROJECTS>SCREAM_Calib,E3SM_RRM</SAVE_TIMING_DIR_PROJECTS>
<CIME_OUTPUT_ROOT>/eagle/E3SM_RRM/$USER/scratch</CIME_OUTPUT_ROOT>
Copy link
Member

Choose a reason for hiding this comment

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

Doe ALCF have a $PROJECT variable we could put here instead of E3SM_RRM ?

Copy link
Member

Choose a reason for hiding this comment

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

Or I guess we can use the PROJECT variable in the xml file.

<SAVE_TIMING_DIR_PROJECTS>SCREAM_Calib,E3SM_RRM</SAVE_TIMING_DIR_PROJECTS>
<CIME_OUTPUT_ROOT>/eagle/E3SM_RRM/$USER/scratch</CIME_OUTPUT_ROOT>
<DIN_LOC_ROOT>/grand/SCREAM_Calib/inputdata</DIN_LOC_ROOT>
<DIN_LOC_ROOT_CLMFORC>/grand/SCREAM_Calib/inputdata/atm/datm7</DIN_LOC_ROOT_CLMFORC>
Copy link
Member

Choose a reason for hiding this comment

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

Fix this as well.

<DIN_LOC_ROOT>/grand/SCREAM_Calib/inputdata</DIN_LOC_ROOT>
<DIN_LOC_ROOT_CLMFORC>/grand/SCREAM_Calib/inputdata/atm/datm7</DIN_LOC_ROOT_CLMFORC>
<DOUT_S_ROOT>$CIME_OUTPUT_ROOT/archive/$CASE</DOUT_S_ROOT>
<BASELINE_ROOT>/eagle/E3SM_RRM/baselines/$COMPILER</BASELINE_ROOT>
Copy link
Member

Choose a reason for hiding this comment

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

We need a better place for this too.

Copy link
Member

@rljacob rljacob left a comment

Choose a reason for hiding this comment

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

Need to change some paths.

@amametjanov amametjanov requested a review from rljacob February 1, 2024 04:12
amametjanov added a commit that referenced this pull request Feb 13, 2024
Add ALCF Polaris to E3SM machines.

[BFB]
@amametjanov amametjanov merged commit 0ad53bb into master Feb 13, 2024
3 checks passed
@amametjanov amametjanov deleted the xyuan/polaris branch February 13, 2024 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants