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

Update files for LANL IC machine chicoma-cpu #5499

Merged
merged 4 commits into from
Mar 9, 2023

Conversation

jonbob
Copy link
Contributor

@jonbob jonbob commented Mar 3, 2023

Updates machine files for chicoma-cpu after the modules were changed and new project space added

[BFB]

@jonbob jonbob added Machine Files BFB PR leaves answers BFB labels Mar 3, 2023
@jonbob jonbob requested a review from darincomeau March 3, 2023 17:50
@jonbob jonbob self-assigned this Mar 3, 2023
Comment on lines +3430 to +3431
<command name="load">PrgEnv-intel/8.3.3</command>
<command name="load">intel-classic/2022.2.1</command>
Copy link
Contributor

Choose a reason for hiding this comment

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

@jonbob, have you been able to build E3SM successfully with this configuration? If so, I'd like to add it to mache (and compass and polaris).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@xylar -- yes, and this is what I gave to Irena for her recent runs.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, okay. I hadn't been paying attention. I assumed she was building with gnu still.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, my bad -- I wasn't paying attention to the specifics of your question. No, I still haven't gotten it to run with intel, but wanted to have the correct module for testing and further work

Copy link
Contributor

Choose a reason for hiding this comment

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

In her build script, she isn't specifying a compiler so I think it's defaulting to gnu still.

Copy link
Contributor

Choose a reason for hiding this comment

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

(Unless that got changed from the version of the build script I saw last.)

Copy link
Contributor

Choose a reason for hiding this comment

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

No, I still haven't gotten it to run with intel, but wanted to have the correct module for testing and further work.

Okay I'll hold off on this until I hear how things go.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, gnu is the default compiler -- along with mpich mpi

Copy link
Member

@darincomeau darincomeau left a comment

Choose a reason for hiding this comment

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

Approved based on successfully running a smoke branch (using default gnu).

Thanks @jonbob !

@jonbob
Copy link
Contributor Author

jonbob commented Mar 7, 2023

@xylar -- are you ok with this going in as-is?

@xylar
Copy link
Contributor

xylar commented Mar 7, 2023

Yes, I wasn't actually asked to review so I was just commenting for my own information.

Copy link
Contributor

@xylar xylar left a comment

Choose a reason for hiding this comment

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

Approving based on Irena Vankova's successful runs.

jonbob added a commit that referenced this pull request Mar 7, 2023
)

Update files for LANL IC machine chicoma-cpu

Updates machine files for chicoma-cpu after the modules were changed and
new project space added

[BFB]
@jonbob
Copy link
Contributor Author

jonbob commented Mar 7, 2023

passes sanity testing -- merged to next

xylar added a commit to xylar/mache that referenced this pull request Mar 8, 2023
@xylar xylar mentioned this pull request Mar 8, 2023
64 tasks
@jonbob jonbob merged commit 92db0e2 into master Mar 9, 2023
@jonbob jonbob deleted the jonbob/machines/update-chicoma-230214 branch March 9, 2023 16:58
@jonbob
Copy link
Contributor Author

jonbob commented Mar 9, 2023

merged to master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB Machine Files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants