Make sure both CI and develoment environments don't include conda MPI #902
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We build the package on HPC using the CI recipe, so it is better if it doesn't use conda's MPI. In addition, we don't want development environments to include MPI because the conda MPI package typically don't work on HPC, where we do a lot of our development and testing.
Finally, this merge also removes some unnecessary
srun
calls tompas_analysis
in the test suite. On many systems, nesting calls tosrun
within a parallel job doesn't work, and given that we may want to callESMF_RegridWeightGen
orncclimo
withsrun
within MPAS-Analysis, it would be better if we don't also callmpas_analysis
unnecessarily withsrun
.