Skip to content

Conversation

@jjz73
Copy link
Collaborator

@jjz73 jjz73 commented Nov 12, 2025

When running Gkeyll with the version of OpenMPI that is currently built by default (v4.1.6), Gkeyll's native eqdskreader tool throws an MPI-related error message. I have tested this when building with v5.0.7 though, and the tool cooperates and generates the .gkyl files containing $\psi$, a necessity for informing our simulation grids.

….1.6 -> 5.0.7) so that Gkeyll's eqdskreader tool works
@manauref
Copy link
Collaborator

manauref commented Nov 12, 2025

Can you make sure the mpi_comm unit tests pass please?

@jjz73
Copy link
Collaborator Author

jjz73 commented Nov 16, 2025

It says that all checks on here have passed, but I checked the log and it doesn't look like it runs the mpi_comm tests in that sequence, it just builds them. I tried running them on my local Gkeyll build and they don't seem to pass yet, but I don't know if they did even when I was running v4.1.6 of MPI, so I'm not sure if it's an issue with how I'm building the code in general.

@manauref
Copy link
Collaborator

you have to run them yourself, try doing this in main

mpirun -np 1 build/core/mctest_mpi_comm -M -E
mpirun -np 2 build/core/mctest_mpi_comm -M -E
mpirun -np 4 build/core/mctest_mpi_comm -M -E
mpirun -np 8 build/core/mctest_mpi_comm -M -E

(with different numbers of processes because there are multiple tests, some use specific number of processes) and then do it again on this branch and make sure the behavior is the same

@jjz73
Copy link
Collaborator Author

jjz73 commented Nov 17, 2025

On both branches, the test mctest_mpi_comm passes with 1, 4, and 8 processes, but fails with 2 processes. The behavior is identical on both branches.

@manauref
Copy link
Collaborator

can you post failure message? or whatever it says please?

@jjz73
Copy link
Collaborator Author

jjz73 commented Nov 18, 2025

Screenshot 2025-11-18 at 10 00 39 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants