Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Check results in tests #42

Open
certik opened this issue Mar 25, 2025 · 1 comment
Open

Check results in tests #42

certik opened this issue Mar 25, 2025 · 1 comment

Comments

@certik
Copy link
Collaborator

certik commented Mar 25, 2025

And in a test like this one: https://github.com/gxyd/c_mpi/blob/main/tests/allreduce.f90, we need to check the result to ensure that:

  • It is computing the correct result
  • It is computing exactly the same answer with any MPI ranks (we'll only test 1, 2, and 4 at the CI).
@certik
Copy link
Collaborator Author

certik commented Mar 25, 2025

Check the result like we do in integration test: "if (abs(result - 55.47) > eps) error stop"

Make sure to print the result too with full double precision accuracy, it should differ as you run it with different MPI ranks within good (machine) accuracy.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant