From 11457c4faed1deb968b8c1c3ce023c3ac54f9fae Mon Sep 17 00:00:00 2001 From: Nathan Goldbaum Date: Fri, 18 Jan 2019 09:10:57 -0600 Subject: [PATCH] clarify rockstar error message about using the wrong number of MPI processes --- yt_astro_analysis/halo_finding/rockstar/rockstar.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/yt_astro_analysis/halo_finding/rockstar/rockstar.py b/yt_astro_analysis/halo_finding/rockstar/rockstar.py index 885b1031..4bbe70fb 100644 --- a/yt_astro_analysis/halo_finding/rockstar/rockstar.py +++ b/yt_astro_analysis/halo_finding/rockstar/rockstar.py @@ -91,9 +91,11 @@ def __init__(self, num_readers, num_writers): else: self.num_writers = min(num_writers, psize) if self.num_readers + self.num_writers + 1 != psize: - mylog.error('%i reader + %i writers + 1 server != %i mpi', - self.num_readers, self.num_writers, psize) - raise RuntimeError + raise RuntimeError( + 'The number of MPI processes (%i) does not equal the ' + 'number of readers (%i) plus the number of writers ' + '(%i) plus 1 server' % ( + self.num_readers, self.num_writers, psize)) def run(self, handler, wg): # Not inline so we just launch them directly from our MPI threads.