llnl / llnl/libROM

Parallel snapshot generation with a custom MPI communicator

Open
#197 2 comments 0 reactions 1 assignee View on GitHub

@dreamer2368 is already working on this.

Since Mar 21, 2023.

enhancement help wanted question
Dominant language
C++
Stars
236
Forks
47
PR merge metrics
No merged PRs in 30d

Description

Currently `BasisReader`, `BasisWriter`, and `BasisGenerator` are initialized with only `MPI_COMM_WORLD`. This has been a limitation, particularly for generating samples in parallel. This happens when multiple samples can be run in parallel, but no communication is needed between each sample, so each sample can have a different communicator or even simply run with one process.

Did anybody come across with this situation, and how did you work around this? Or should we change the initializations so that each class can be initialized with a custom MPI Communicator?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.