[Custom Issue]: Add dummy functions for MPI IO operations
Open
@mo-mglover is already working on this.
Since Jun 20, 2025.
enhancement
- Dominant language
- C++
- Stars
- 6
- Forks
- 11
- Avg merge
- 18h 7m
- Merged PRs (30d)
- 1
Description
Contact Details
No response
Are there any linked Issues or Pull Requests?
Yes! PR #197
Brief description
A bit of outstanding technical debt from identified by @andrewcoughtrie as part of his review of the PR:
My thoughts are both that this shouldn't be kept in the mpi_context.cpp file it would probably be preferable to wrap/dummy the actual MPI calls and have the actually use in the single.cpp file. This would keep the MPI usage contained away from our usage in the writers.
Further details of the issue.
A big chunk of code in mpi_context.cpp is currently ifdefed out when MPI isn't available. This should be replaced with some dummy functions to make it consistent with the approach used elsewhere in the code base.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.