MRChemSoft / mrcpp

MultiResolution Computation Program Package
GNU Lesser General Public License v3.0
12 stars 19 forks source link

Encapsulate MPI #151

Closed stigrj closed 4 years ago

stigrj commented 4 years ago

Has been tested to work for:

stigrj commented 4 years ago

This means that a fully parallel (--omp --mpi) MRCPP can now be used in clients with any level of parallelization; serial, omp, mpi or hybrid (as long as it's correctly implemented in the client, ofc)

codecov[bot] commented 4 years ago

Codecov Report

Merging #151 into master will not change coverage. The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #151   +/-   ##
=======================================
  Coverage   67.11%   67.11%           
=======================================
  Files         172      172           
  Lines       12153    12153           
=======================================
  Hits         8156     8156           
  Misses       3997     3997           
Impacted Files Coverage Δ
src/trees/SerialTree.cpp 98.27% <ø> (ø)
src/utils/Printer.cpp 3.64% <ø> (ø)
src/utils/mpi_utils.cpp 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f093753...ff31c6e. Read the comment docs.

MinazoBot commented 4 years ago
2 Warnings
:warning: There are code changes, but no corresponding tests. Please include tests if this PR introduces any modifications in behavior.
:warning: Consider adding supporting documentation to this change. Documentation sources can be found in the doc directory.

Generated by :no_entry_sign: Danger