Refactor the communicator classes to be more consistent with each other (MPI and Threads equivalent operations have the same arguments) and remove if use_mpi statements for easier to read workflows.
Approach
Add tests for all of the MPI and Pool methods.
Use only MPI-inclusive communicator patterns from Comm, and use a placeholder MPI when MPI is not available.
Pre-Merge Checklists
Submitter
[ ] Write a helpfully descriptive pull request title.
[ ] Organize changes into logically grouped commits with descriptive commit messages.
[ ] Document all new functions.
[ ] Click 'details' on the readthedocs check to view the updated docs.
[ ] Write tests for new functions or explain why they are not needed.
[ ] Address any complaints from pep8speaks.
Reviewer
[ ] Actually read all of the code.
[ ] Run the new code yourself; the included tests should make this easy.
[ ] Write a summary of the changes as you understand them.
Purpose
Refactor the communicator classes to be more consistent with each other (MPI and Threads equivalent operations have the same arguments) and remove
if use_mpistatements for easier to read workflows.Approach
Add tests for all of the MPI and Pool methods. Use only MPI-inclusive communicator patterns from Comm, and use a placeholder MPI when MPI is not available.
Pre-Merge Checklists
Submitter
Reviewer