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 the quick-start for developers #209

Merged
merged 4 commits into from
Mar 7, 2025

Conversation

xylar
Copy link

@xylar xylar commented Mar 7, 2025

This merge includes new paths to metis/parmetis and updated names for the compilers on Frontier.

Checklist

  • Documentation:
    • Developer's Guide has been updated
    • Documentation has been built locally and changes look as expected

@xylar xylar added the documentation Improvements or additions to documentation label Mar 7, 2025
@xylar xylar requested a review from grnydawn March 7, 2025 15:14
@xylar xylar self-assigned this Mar 7, 2025
This merge includes new paths to metis/parmetis and updated names
for the compilers on Frontier.
@xylar xylar force-pushed the omega/docs/update-machines branch from 8890f72 to 8f1d36b Compare March 7, 2025 15:28
We decided not to support them, as there were problems compiling
Omega with them.
@xylar
Copy link
Author

xylar commented Mar 7, 2025

@grnydawn, I'm working on building this locally to make sure I didn't mess up the formatting of the table -- it looks okay in code at least. Could you quickly verify that the compiler names all look correct?

@grnydawn
Copy link

grnydawn commented Mar 7, 2025

@xylar The compiler names are all correct. Omega does not support craygnu-hipcc because it is primarily used for Scream experiments. You may remove this compiler from the list.

@xylar
Copy link
Author

xylar commented Mar 7, 2025

@grnydawn, good to know about craygnu-hipcc. I will remove it from mache and Polaris, too.

We will not support this in Omega, it's just for EAMxx
@xylar
Copy link
Author

xylar commented Mar 7, 2025

(By the way, CTests pass with craygnu-hipcc. I tested them in E3SM-Project/polaris#282.)

@grnydawn
Copy link

grnydawn commented Mar 7, 2025

Thanks for the info. craygnu-hipcc was created by Trey at ORNL and includes the latest software stack. craygnu-mphipcc shares the same software stack as craygnu-hipcc, with some differences in CMake settings.

@xylar
Copy link
Author

xylar commented Mar 7, 2025

Copy link

@grnydawn grnydawn left a comment

Choose a reason for hiding this comment

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

Approved.

@xylar
Copy link
Author

xylar commented Mar 7, 2025

I'm ignoring Codacy. I think it's "errors" are BS.

@xylar
Copy link
Author

xylar commented Mar 7, 2025

@grnydawn, thanks for the review!

@xylar xylar merged commit d8d0f3c into E3SM-Project:develop Mar 7, 2025
2 of 3 checks passed
@xylar xylar deleted the omega/docs/update-machines branch March 7, 2025 16:27
@rljacob
Copy link
Member

rljacob commented Mar 7, 2025

Hang on. I don't think that's true about craygnu-hipcc being only for EAMxx.

@xylar
Copy link
Author

xylar commented Mar 7, 2025

@rljacob, I'll leave that to you and @grnydawn to sort out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants