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

Add pe layout for G compset with MARBL and WW3 #233

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mnlevy1981
Copy link
Collaborator

Use the same layout as G compset with WW3 but not MARBL, except I bump NTASKS_OCN up to 2560.

I also fixed a bug where G compset with MARBL and SWAV was letting 2304 cores sit idle (ROOTPE_OCN was 2560 instead of 256)

fixes #232

Use the same layout as G compset with WW3 but not MARBL, except I bump
NTASKS_OCN up to 2560.

I also fixed a bug where G compset with MARBL and SWAV was letting 2304 cores
sit idle (ROOTPE_OCN was 2560 instead of 256)
@mnlevy1981
Copy link
Collaborator Author

For the ROOTPE_OCN bug, it looks like I had the right value for TL319 atmosphere grids and the typo in the T62 atmosphere grid section, then we kept the wrong value when removing support for T62 in #215

Copy link
Member

@alperaltuntas alperaltuntas left a comment

Choose a reason for hiding this comment

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

Note that in about a week, I'll submit a PR to FMS and MOM6 to enable parallel IO, which may render these PE layouts outdated.

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.

PE layout when MARBL and WW3 are both present is poor
2 participants