Skip to content

Conversation

@opotowsky
Copy link
Member

What is the change? Why is it being made?

ARMI has a try/except for the from mpi4py import MPI that has historically worked for us downstream. The exception, ImportError, has captured all our needs up until now.

Now (mpi4py 4.1.1 upgrade to go with the pkl5 change), that import has a different exception: RuntimeError (RuntimeError: cannot load MPI library). We are failing upon all invocations of armi as a result, unless we have an OMPI module loaded.

SCR Information

Change Type: features

One-Sentence Rationale: Upgrading mpi4py caused a new error; this update handles that situation.

One-line Impact on Requirements: NA


Checklist

@opotowsky opotowsky added the enhancement New feature or request label Dec 19, 2025
Copy link
Member

@john-science john-science left a comment

Choose a reason for hiding this comment

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

@opotowsky Approved. And you have Merge power.

Co-authored-by: John Stilley <1831479+john-science@users.noreply.github.com>
@opotowsky opotowsky merged commit f085bd5 into main Dec 23, 2025
12 checks passed
@opotowsky opotowsky deleted the new-year-new-exception branch December 23, 2025 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants