[BUG] Fix #61
Fixes a basic mistake in ParallelExecutor
that would cause the executor to report completion as soon as it receives the first status message from one of its children (i.e. immediately).
... I guess a couple of basic unit tests wouldn't hurt ...
@armdanie The example given in #61 (closed) reports the issue specifically on SerialExecutor
. Could you double check that it wasn't in fact observed on ParallelExecutor
? SerialExecutor
should really only post_process()
when all children are done:
executor.py#L150-154