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 to version of MOM5 that doesn't require unused WOMBAT inputs #12

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

dougiesquire
Copy link
Contributor

Closes #9

The version of MOM5 only requires input files for WOMBAT gas exchange when they are used
Copy link

🚀 Deploying access-esm1.5 2024.05.1 as prerelease pr12-1

Details and usage instructions

This access-esm1.5 model will be deployed as:

  • 2024.05.1 as a Release (when merged).
  • pr12-1 as a Prerelease (during this PR).

This Prerelease is accessible on Gadi using:

module use /g/data/vk83/prerelease/modules/access-models/
module load access-esm1p5/pr12-1

where the binaries shall be on your $PATH.
This Prerelease is also accessible on Gadi via /g/data/vk83/prerelease/apps/spack/0.21/spack in the access-esm1p5-pr12-1 environment.

🛠️ Using: spack-packages `2024.07.10`, spack-config `2024.07.05`
Details

It will be deployed using:

If this is not what was expected, commit changes to config/versions.json.

Copy link
Member

@CodeGat CodeGat left a comment

Choose a reason for hiding this comment

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

Makes sense to me. Does this need to be done before the workshop @dougiesquire?

@dougiesquire
Copy link
Contributor Author

I think @aidanheerdegen was keen for it, and I quote:

even better if it was before the workshop (NO PRESSURE!)

@CodeGat
Copy link
Member

CodeGat commented Aug 28, 2024

Well, if it's been tested and seems to do what it says on the tin, give it a merge :)

@CodeGat
Copy link
Member

CodeGat commented Aug 28, 2024

Note: the deployment might seem to fail but it is just the end part where the build provenance database hasn't had ACCESS-NRI/build-cd#113 merged yet

@dougiesquire dougiesquire merged commit 6ba5764 into main Aug 28, 2024
12 checks passed
@dougiesquire dougiesquire deleted the issue9-mom5-update branch August 28, 2024 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include MOM5 code changes for required input files
2 participants