Skip to content

Namelist and external module (clm,ccs_config,cmeps) updates for ne1024 grid#79

Merged
sjsprecious merged 1 commit intostormspeedfrom
ne1024_defaults
Mar 5, 2026
Merged

Namelist and external module (clm,ccs_config,cmeps) updates for ne1024 grid#79
sjsprecious merged 1 commit intostormspeedfrom
ne1024_defaults

Conversation

@jtruesdal
Copy link
Collaborator

These defaults are needed to run ne1024 F Case. The three local externals are in place until the ne1024 experiments are validated.

@jtruesdal jtruesdal requested review from Copilot and sjsprecious March 5, 2026 16:40
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Collaborator

@sjsprecious sjsprecious left a comment

Choose a reason for hiding this comment

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

Thanks @jtruesdal for working on the update. They look good to me.

Once you have the new tags for the external modules, I think this PR is ready to go.

@sjsprecious sjsprecious self-assigned this Mar 5, 2026
@sjsprecious sjsprecious added enhancement New feature or request ready_for_ci This PR is ready for CI tests on CIRRUS cloud and removed ready_for_ci This PR is ready for CI tests on CIRRUS cloud labels Mar 5, 2026
@jtruesdal
Copy link
Collaborator Author

All the tags exist for the externals. I am running with them for the ne1024 tests. Once I get this running and make sure all the defaults in the externals are OK I'll create the PRs for the official repos. I'll wait for the CI tests to pass and if you approve I'll go ahead and merge up.

Copy link
Collaborator

@sjsprecious sjsprecious left a comment

Choose a reason for hiding this comment

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

Thanks @jtruesdal . It seems all the CI tests are passed. Do you want to update the official tags in a separate PR or in this PR?

@jtruesdal
Copy link
Collaborator Author

Yes, I would like this PR in before the official tags. Thanks for merging in the clubb changes first, I forgot about that. Let me know if there are any issues with that and I'll do it on the command line. This group of changes checkpoint a state where clubb/mf and the ne1024 mods are bfb with CAM regressions. My next PR will bring all these updates to the latest development tag. There too, stormspeed clubb tests should be bfb with the CAM regression suite. I'm hoping to have that also done before the reservation next week, and will work on that as soon as this PR is in. There will most likely be a clean up PR, tweaking the out of the box defaults and external mods after I can debug with the reservation slot. Our collaborators should be able to use stormspeed for high res testing after the cleanup PR. Then three separate PR's to the clm, cmeps, and ccs_config repositories to get the official tags for those externals. I think we have a much better chance of being able to use those new official externals in stormspeed since it will also be up to the head of development.

@sjsprecious
Copy link
Collaborator

Thanks @jtruesdal for your detailed explanation. That sounds good and I will go ahead and merge your PR.

I also noticed that the latest CAM development branch had some code fixes for the latest ifx compiler. It will be nice to bring them into StormSPEED as well before I create a newer intel image.

@sjsprecious sjsprecious merged commit f18f915 into stormspeed Mar 5, 2026
15 of 26 checks passed
@sjsprecious sjsprecious deleted the ne1024_defaults branch March 5, 2026 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ready_for_ci This PR is ready for CI tests on CIRRUS cloud

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants