-
Notifications
You must be signed in to change notification settings - Fork 358
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 logic to catch issue when partition file assigns 0 cells to a pe #6321
Conversation
|
Testing with a oQU240wLI configuration that pointed to a manipulated graph file both replicated the original issue and showed that this fix allows mpas-ocean to successfully complete |
@jonbob Thanks for fixing this. The logic makes sense to me. Would you like me to do any additional testing before approving? |
Thanks @cbegeman -- I don't think any testing is necessary, unless you would like to try it. I'm redoing the test I had setup just to make sure this solution works exactly as in the PR. And I'll do more testing to make sure it doesn't change answers on any regular E3SM tests before I merge it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add logic to catch issue when partition file assigns 0 cells to a pe When mpas-ocean runs with no cells assigned to a processor by a graph file and has landIcePressureOn, the run aborts. This fixes that situation by adding a check for the number of cells to the logic that causes the abort. Fixes: #6320 [BFB]
Passes:
merged to next |
This merge updates the E3SM-Project submodule from [93e511d](https://github.com/E3SM-Project/E3SM/tree/93e511d) to [31e0924](https://github.com/E3SM-Project/E3SM/tree/31e0924). This update includes the following MPAS-Ocean and MPAS-Frameworks PRs (check mark indicates bit-for-bit with previous PR in the list): - [ ] (ocn) E3SM-Project/E3SM#6256 - [ ] (ocn) E3SM-Project/E3SM#6224 - [ ] (ocn) E3SM-Project/E3SM#6270 - [ ] (ocn) E3SM-Project/E3SM#6293 - [ ] (ocn) E3SM-Project/E3SM#6321 - [ ] (ocn) E3SM-Project/E3SM#6262 - [ ] (ocn) E3SM-Project/E3SM#6300 - [ ] (ocn) E3SM-Project/E3SM#6334 - [ ] (ocn) E3SM-Project/E3SM#6371 - [ ] (ocn) E3SM-Project/E3SM#6288
When mpas-ocean runs with no cells assigned to a processor by a graph file and has landIcePressureOn, the run aborts. This fixes that situation by adding a check for the number of cells to the logic that causes the abort.
Fixes: #6320
[BFB]