Uh oh!
There was an error while loading. Please reload this page.
add support for mpi4py - #190
Conversation
Uh oh!
There was an error while loading. Please reload this page.
| elif with_distributed and isinstance(ex, distributed.cfexecutor.ClientExecutor): | ||
| return sum(n for n in ex._client.ncores().values()) | ||
| elif with_mpi4py and isinstance(ex, mpi4py.futures.MPIPoolExecutor): | ||
| return mpi4py.MPI.COMM_WORLD.size - 1 |
There was a problem hiding this comment.
ex.bootup() # wait until all workers are up and runningreturnexecutor._pool.size# not public API!There was a problem hiding this comment.
That's better, does ex._pool.size work before all the workers are up and running? Because Adaptive can handle scaling of the pool size.
jbweston
commented
May 1, 2019
does this "just work"? Isn't there some extra bits needed for launching workers? We should probably document this somehow... |
@jbweston I'll add some more details to the docs later. In a nutshell, it works when calling your Python script like: or in a SLURM job |
dalcinl
commented
May 3, 2019
In your desktop or laptop, it can also work like this: Or you can pass In this case, the 15 workers will be MPI-spawned at runtime. I consider this the preferred way of using |
dalcinl
commented
May 3, 2019
You are hurting my feelings 😉 |
basnijholt
commented
May 6, 2019
jbweston
commented
May 7, 2019
LGTM |
No description provided.