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

Do we really want to vary PE layouts based on atmospheric grid? #95

Open
mnlevy1981 opened this issue Oct 26, 2021 · 1 comment
Open

Comments

@mnlevy1981
Copy link
Collaborator

mnlevy1981 commented Oct 26, 2021

On my MARBL branch, I ran into an issue with getting unexpected PE layouts because I had modified the <grid name="a%T62.+oi%tx0.66v1"> section of config_pes.xml but hadn't added support for MOM6%MARBL compsets in the <grid name="a%TL319.+oi%tx0.66v1"> section... but it strikes me as odd that we would use different layouts for those two grids. In fact, it looks like these two sections are set up to give identical layouts*. Should they be combined into a single <grid name="oi%tx0.66v1"> section?

* I might have gotten a little lost going back and forth between the two sections, but if I'm reading the file right then on cheyenne MOM gets 864 tasks in all four cases (T62 + C compset, T62 + G compset, TL319 + C compset, TL319 + G compset); the rest of the components share 72 tasks for C compsets or 108 tasks for G compsets, regardless of atm grid.

@alperaltuntas
Copy link
Member

Should they be combined into a single section?

I agree.

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

No branches or pull requests

2 participants