Skip to content

Commit

Permalink
rearrange manager warning message placement, mention Open MPI vs. mpich
Browse files Browse the repository at this point in the history
  • Loading branch information
jlnav committed Jul 11, 2023
1 parent 1534b68 commit 87e9cec
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions libensemble/libE.py
Original file line number Diff line number Diff line change
Expand Up @@ -387,18 +387,18 @@ def libE_mpi_manager(mpi_comm, sim_specs, gen_specs, exit_criteria, persis_info,
"""Manager routine runs on rank 0."""
from libensemble.comms.mpi import MainMPIComm

if Executor.executor is not None:
if isinstance(Executor.executor, MPIExecutor) and libE_specs["comms"] == "mpi":
logger.manager_warning(
"WARNING: Nested MPI-workflow detected. User initialized both an MPI runtime and an MPI Executor.\n"
+ " Expect complications if not using an `MPICH` MPI distribution.\n"
)

if not libE_specs["disable_log_files"]:
exit_logger = manager_logging_config(specs=libE_specs)
else:
exit_logger = None

if Executor.executor is not None and isinstance(Executor.executor, MPIExecutor):
logger.manager_warning(
"WARNING: Nested MPI-workflow detected. User initialized both an MPI runtime and an MPI Executor.\n"
+ " Expect complications if using Open MPI."
+ " An MPICH-derived MPI distribution is recommended instead.\n"
)

def cleanup():
"""Process cleanup required on exit"""
if exit_logger is not None:
Expand Down

0 comments on commit 87e9cec

Please sign in to comment.