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

Support update to MOM6 #468

Merged
merged 4 commits into from
Nov 9, 2021
Merged

Conversation

sanAkel
Copy link
Contributor

@sanAkel sanAkel commented Oct 13, 2021

There is only one update in this PR and it relates to MOM6 ocean model - only (hence MOM5 and nothing is touched).

Update:
The CMakeLists.txt to build MOM6 has been updated to include additional files and hence with this PR, we can work with more recent releases of MOM6 such as geos/v2.0.2.

Dependency:
@sdrabenh and @mathomp4 I will Put in another PR to update MOM6 to geos/v2.0.2. Please see: GEOS-ESM/GEOSgcm#344 Thank you!

sanAkel added 2 commits October 12, 2021 20:32
…hich is same as in [geos/v2.0.2](https://github.com/GEOS-ESM/MOM6/tree/geos/v2.0.2), 2. From @mathomp4, a switch to either FMS1 or FMS2 infra has been added, default is former (until GEOS as a whole switches to FMS2); MOM6 coupled model with FMS2 has been tested to work. For future reference, most of the list of CMakeLists.txt has been produced by a Concatenation of output from following two commands:
@sanAkel sanAkel added enhancement New feature or request 0 diff The changes in this pull request have verified to be zero-diff with the target branch. Contingent - DNA These changes are contingent on other PRs (DNA=do not approve) labels Oct 13, 2021
@sanAkel sanAkel requested review from mathomp4 and a team October 13, 2021 00:50
@sanAkel sanAkel self-assigned this Oct 13, 2021
sanAkel added a commit to GEOS-ESM/GEOSgcm that referenced this pull request Oct 13, 2021
This PR is related to update of MOM6 release: `geos/v.2.0.2` and GEOS-ESM/GEOSgcm_GridComp#468
@mathomp4
Copy link
Member

For @sdrabenh, this is another chicken-and-egg PR. It can't succeed without mom6 2.0.2, but the CI already gets 2.0.1 from main before it runs here. (And likewise, 2.0.2 can't build without this PR.)

The fun of changed interfaces!

@sdrabenh sdrabenh removed the Contingent - DNA These changes are contingent on other PRs (DNA=do not approve) label Nov 9, 2021
@sdrabenh sdrabenh merged commit 1275745 into develop Nov 9, 2021
@sdrabenh sdrabenh deleted the feature/sanAkel/mom6-maintenance-12Oct2021 branch November 9, 2021 18:21
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. enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants