-
Notifications
You must be signed in to change notification settings - Fork 371
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
Change Ocean conservationCheck output frequency #6412
Change Ocean conservationCheck output frequency #6412
Conversation
TestingThis PR was found to increase total compute time by 0.5% on perlmutter cpu for a 1-month B case. |
should be "1 day", right? |
@xylar Yes, of course. Thanks for catching that! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm approving by inspection and based on @cbegeman's testing.
@jonbob The test that I ran ( |
#6412) Change Ocean conservationCheck output frequency For reasons that are not entirely clear, we have found that the ocean's conservation analysis member outputs with the wrong start time when the output frequency is 1 month. This issue is fixed when the frequency is increased (1 day and 10 days were tested). Here, we change the default output frequency of the conservationCheckOutput stream to 1 day. [BFB]
passes:
merged to next |
Thanks for the testing @jonbob! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved based on visual inspection and testing
merged to master |
This merge updates the E3SM-Project submodule from [8939709](https://github.com/E3SM-Project/E3SM/tree/8939709) to [752f281b15](https://github.com/E3SM-Project/E3SM/tree/752f281b15). This update includes the following MPAS-Ocean and MPAS-Frameworks PRs (check mark indicates bit-for-bit with previous PR in the list): - [ ] (ocn) E3SM-Project/E3SM#6412 - [ ] (ocn) E3SM-Project/E3SM#6434 - [ ] (ocn) E3SM-Project/E3SM#6420
For reasons that are not entirely clear, we have found that the ocean's conservation analysis member outputs with the wrong start time when the output frequency is 1 month. This issue is fixed when the frequency is increased (1 day and 10 days were tested). Here, we change the default output frequency of the
conservationCheckOutput
stream to 1 day.Fixes MPAS-Dev/MPAS-Analysis#999