Skip to content

Update MOM6 ocean layouts for o720 and o2880 resolutions - #137

Merged
mathomp4 merged 4 commits into
developfrom
feature/update-mom6-layouts
Aug 28, 2026
Merged

Update MOM6 ocean layouts for o720 and o2880 resolutions#137
mathomp4 merged 4 commits into
developfrom
feature/update-mom6-layouts

Conversation

@mathomp4

@mathomp4 mathomp4 commented Aug 21, 2026

Copy link
Copy Markdown
Member

Description

This PR aligns MOM6 ocean layouts at 1920 ranks. It must be used with GEOS-ESM/GEOSgcm_App#910 so the coupled AGCM and ocean components use matching rank counts.

This enables every supported C180 MOM6 configuration to share the same 20x96 atmospheric layout.

Ocean grid NXxNY Local tile Ranks
o720 (720x576) 40x48 18x12 1920
o1440 (1440x1080) 48x40 30x27 1920
o2880 (2880x2240) 48x40 60x56 1920

Validated: a C180/o720 coupled run completed successfully.

Should be taken in concert with:

@mathomp4 mathomp4 self-assigned this Aug 21, 2026
@mathomp4 mathomp4 added 0 diff dataocean Zero-diff for the dataocean model 0 diff coupled Zero-diff for the coupled model 0 diff dataatm Zero-diff for the data atm model labels Aug 21, 2026
@mathomp4
mathomp4 marked this pull request as ready for review August 26, 2026 14:39
@mathomp4
mathomp4 requested a review from a team as a code owner August 26, 2026 14:39

@sinakhani sinakhani left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I tested the new layouts for o720 and o2880 configurations -- worked fine. Thank you @mathomp4.

@mathomp4
mathomp4 merged commit 129c4b6 into develop Aug 28, 2026
24 of 25 checks passed
@sinakhani

Copy link
Copy Markdown
Collaborator

Hi @mathomp4. Please could you also update NX, NY, and OGCM.NX, OGCM.NY in AGCM.rc based on these new oce/atm layouts. Otherwise, the exp will fail because of a layout mismatch.

@mathomp4

mathomp4 commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

@sinakhani Those should be updated when taken with GEOS-ESM/GEOSgcm_App#910

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff coupled Zero-diff for the coupled model 0 diff dataatm Zero-diff for the data atm model 0 diff dataocean Zero-diff for the dataocean model

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants