llnl / llnl/MACSio

Eliminate HAVE_MPI conditional compilation

Open
#18 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
34
Forks
16
PR merge metrics
No merged PRs in 30d

Description

We made the MPI code conditionally compiled originally on the idea that maybe multiple forms of parallelism would be supported here. But, do we expect to be doing anything useful at large scale without MPI? If not, having MPI conditionally compiled only complicates the code without much benefit and it should be removed.

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the repository for HAVE_MPI conditional-compilation guards and read the surrounding MPI implementation and build configuration. Confirm the full scope of the guards, then run the repository's existing build or test commands; done means the conditional MPI paths are removed without breaking the MPI-enabled build.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
hpc
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.