Remove MPI runtime requirement for ParMETIS test #269
Remove MPI runtime requirement for ParMETIS test #269ocaisa wants to merge 1 commit intoelemental:masterfrom
Conversation
The test run of ParMETIS is done without the use of `MPIEXEC_EXECUTABLE` and this can lead to false negatives with some MPI implementations. This change further trivialises the test code to avoid this issue (and, given the `FIXME`, is equivalent).
|
Please ignore the Travis failure: it was just a timeout. I am not sure of the utility of removing the MPI calls (MPI is required for ParMETIS), but adding an explicit ParMETIS call would be a much-needed addition. |
|
Yes, MPI is required for ParMETIS but the existing test does not launch using which is purely because the compiled code was run directly rather than via |
|
I am no longer maintaining Elemental (I quit my job and am starting up new open source scientific computing projects). I prefer to know less about CMake than I do and have switched to Meson. |
|
Ok, no problem ;) |
The test run of ParMETIS is done without the use of
MPIEXEC_EXECUTABLEand this can lead to false negatives with some MPI implementations. This change further trivialises the test code to avoid this issue (and, given theFIXME, is equivalent).