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 pre-industrial configuration with latest payu updates #42

Merged
merged 10 commits into from
Aug 9, 2024

Conversation

blimlim
Copy link

@blimlim blimlim commented Aug 7, 2024

This pull request completes the pre-industrial half of #40. It modifies the configuration with the following changes:

  • Remove um_env.py file now that it has been replaced by the um_env.yaml file.
  • Remove unused atmosphere configuration files Remove unused configuration files from atmosphere directory #27.
  • Add/update the following settings in the config.yaml file:
    • Add commented out shortpath and project settings
    • Add sync settings with default false setting. Add comment explaining compatibility with postscripts
    • Add restart_freq setting with default value of 5.
    • Add runlog and platform settings with default values.
    • Add manifest: reproduce: exe: True as in the OM2 configurations.

It would be great to get a check over these changes, especially to confirm that the default values for several of the new config.yaml settings are suitable for ESM1.5.

blimlim added 3 commits August 6, 2024 16:35

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
@blimlim blimlim requested a review from aidanheerdegen August 7, 2024 00:58
@blimlim
Copy link
Author

blimlim commented Aug 7, 2024

I'll wait until #38 is merged and update this branch with the changes before trying to merge.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
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.

Some changes, some comments and some questions.

blimlim and others added 4 commits August 8, 2024 14:59

Partially verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
use updated module path for model executable

Co-authored-by: Aidan Heerdegen <aidan.heerdegen@anu.edu.au>

Partially verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
Remove platform settings

Co-authored-by: Aidan Heerdegen <aidan.heerdegen@anu.edu.au>

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
@blimlim blimlim requested a review from aidanheerdegen August 9, 2024 01:06

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
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.

Sorry! One more change required it seems.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
@blimlim blimlim requested a review from aidanheerdegen August 9, 2024 06:22
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.

All good to merge, thanks @blimlim!

@blimlim
Copy link
Author

blimlim commented Aug 9, 2024

Thanks for the review! There will be a similar one soon with the same changes for the historical configuration. I've added in all the changes from the discussion here and so hopefully that one won't be too involved!

@blimlim blimlim merged commit a8ac17d into pre-industrial Aug 9, 2024
@blimlim blimlim deleted the 40-update-pre-industrial-payu-changes branch August 9, 2024 06:33
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.

None yet

2 participants