Closed chowland closed 1 year ago
You're right, it's a documentation issue. It should be MPITopology{2}(comm_cart)
as you noticed. Thanks for catching that.
Note that in the docs for the MPITopology
type the right constructor is used:
comm = MPI.COMM_WORLD
@assert MPI.Comm_size(comm) == 8
pdims = (4, 2)
comm_cart = MPI.Cart_create(comm, pdims)
topology = MPITopology{2}(comm_cart)
I've noticed that the usage of
MPITopology
with a pre-existing Cartesian communicator doesn't work following the documentation. Simply passinggives an error, which is fixed by specifying
I'm not sure whether this is intended (and should then be changed in the docs) or whether another method for
MPITopology
should be defined to make things work as suggested by the documentation