Skip to content

Commit

Permalink
#378 multiprocessing::Context::broadcast() has no effect if MPI is di…
Browse files Browse the repository at this point in the history
…sabled

Signed-off-by: Dimitri Baron <[email protected]>
  • Loading branch information
barondim authored and gautierbureau committed Dec 4, 2024
1 parent 2cb1736 commit d045794
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions sources/Common/DYNMultiProcessingContext.h
Original file line number Diff line number Diff line change
Expand Up @@ -99,9 +99,8 @@ class Context {
static Context* instance_; ///< Unique instance
static bool finalized_; ///< Instance is already finalized

#ifdef _MPI_

public:
#ifdef _MPI_
/**
* @brief Gather all data into root rank
*
Expand All @@ -113,6 +112,7 @@ class Context {
void gather(const T& data, std::vector<T>& recvData) const {
gatherImpl(Tag<T>(), data, recvData);
}
#endif

/**
* @brief Broadcast data from root rank to all process
Expand All @@ -122,9 +122,12 @@ class Context {
*/
template<class T>
void broadcast(T& data) const {
#ifdef _MPI_
broadcastImpl(Tag<T>(), data);
#endif
}

#ifdef _MPI_
/**
* @brief Retrieve the rank of the current process
*
Expand Down

0 comments on commit d045794

Please sign in to comment.