hnn-core
hnn-core copied to clipboard
MPIBackend silently works even if MPI is not available
Not exactly sure if it's intended behavior. If you do:
with MPIBackend(n_procs=3):
dpls = simulate_dipole(net, n_trials=1)
and mpi4py
is not installed, it should throw an ImportError
rather than resorting to joblibs
backend
Seems a good idea to throw an exception. Also a UserWarning
if only 1 processor is given to MPIBackend (which will run simulation using joblibs
). I think this might catch an issue on CI where the cloud server only has 1 processor, so run_hnn_core_fixture
may actually not run MPIBackend
when the test inteded to.
I think HNN GUI could be adapted to catch the ImportError
and run the simulation with the joblibs
backend. Just need changes on that side too...
Is this issue currently open?
@simar5244 yes it is!