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

025deg jra55 iaf #89

Merged
merged 8 commits into from
Apr 18, 2024
Merged

Conversation

anton-seaice
Copy link
Contributor

@anton-seaice anton-seaice commented Apr 15, 2024

Contributes to #17

This adds 0.25 degree IAF

@anton-seaice anton-seaice force-pushed the update-025deg_jra55_iaf-iss17 branch from 5003014 to 411611b Compare April 15, 2024 01:36
@anton-seaice anton-seaice force-pushed the update-025deg_jra55_iaf-iss17 branch from d4e4c66 to 615969a Compare April 15, 2024 03:27
metadata.yaml Outdated Show resolved Hide resolved
@anton-seaice anton-seaice changed the title Update 025deg jra55 iaf for access-nri build 025deg jra55 iaf Apr 17, 2024
@anton-seaice anton-seaice marked this pull request as ready for review April 17, 2024 05:50
Copy link
Member

@aidanheerdegen aidanheerdegen left a comment

Choose a reason for hiding this comment

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

Missed a couple of the exe paths needing to be updated.

config.yaml Outdated Show resolved Hide resolved
config.yaml Outdated Show resolved Hide resolved
Co-authored-by: Aidan Heerdegen <aidan.heerdegen@anu.edu.au>
Copy link
Member

@aidanheerdegen aidanheerdegen left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@aidanheerdegen aidanheerdegen merged commit e0cf97d into dev-025deg_jra55_iaf Apr 18, 2024
7 of 8 checks passed
@aidanheerdegen aidanheerdegen deleted the update-025deg_jra55_iaf-iss17 branch April 18, 2024 06:06
@anton-seaice anton-seaice restored the update-025deg_jra55_iaf-iss17 branch April 18, 2024 06:11
@anton-seaice
Copy link
Contributor Author

We missed the manifests :). Shall i revert the merge or make a new PR or other ?

@anton-seaice anton-seaice mentioned this pull request Apr 18, 2024
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.

3 participants