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 MOM6 Overrides to match current MOM6 results #80

Merged
merged 2 commits into from
Aug 21, 2024

Conversation

mathomp4
Copy link
Member

With the merge of mom-ocean/MOM6#1631 into our MOM6 fork in MOM6 geos/v3.2 without any changes, the answers will change.

So, until these can be tested, we set:

USE_HUYNH_STENCIL_BUG = True
EPBL_ANSWER_DATE = 20231231

in the MOM_override files.

With the merge of mom-ocean/MOM6#1631 into
our MOM6 fork in [MOM6 geos/v3.2](https://github.com/GEOS-ESM/MOM6/releases/tag/geos%2Fv3.2)
without any changes, the answers will change.

So, until these can be tested, we set:
```
USE_HUYNH_STENCIL_BUG = True
EPBL_ANSWER_DATE = 20231231
```

in the `MOM_override` files.
@mathomp4 mathomp4 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Aug 20, 2024
@mathomp4 mathomp4 self-assigned this Aug 20, 2024
@mathomp4
Copy link
Member Author

NOTE: I think there might be more needed here to work with the latest MOM6. So I'm blocking.

@mathomp4 mathomp4 added the Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs label Aug 20, 2024
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: 0 diff, Contingent - DNA

@mathomp4 mathomp4 marked this pull request as ready for review August 20, 2024 18:27
@mathomp4 mathomp4 requested a review from a team as a code owner August 20, 2024 18:27
@mathomp4 mathomp4 merged commit 90077a9 into develop Aug 21, 2024
1 of 4 checks passed
@mathomp4 mathomp4 deleted the feature/mathomp4/add-mom6-32-overrides branch August 21, 2024 12:06
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: 0 diff, Contingent - DNA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch. Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant