Closed wehs7661 closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.56%. Comparing base (
a0f93b7
) to head (0c8a668
).:exclamation: Current head 0c8a668 differs from pull request most recent head 3aa87c2. Consider uploading reports for the commit 3aa87c2 to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
In this PR, we complete the work in
test_mpi_func.py
, which tests functions that use MPI, including_run_grompp
,_run_mdrun
, andrun_REXEE
. Except for the tests itself, some key things were done, includingcomm.barrier
, which presumably prevented some (uninformative) errors fromMPI_Abort()
.setup.cfg
by addingparallel=True
, which prevented the "internal errors" as a result of race conditions with file access between MPI processes.coverage
instead ofpytest
, ascoverage
seems to work better with MPI, especially with the configuration file explicitly passed.