MRChemSoft / mrcpp

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

Use std::array in NodeIndex #162

Closed stigrj closed 3 years ago

codecov[bot] commented 3 years ago

Codecov Report

Merging #162 (b2a26a7) into master (504f03c) will increase coverage by 0.07%. The diff coverage is 97.91%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
+ Coverage   67.19%   67.27%   +0.07%     
==========================================
  Files         169      169              
  Lines       11777    11804      +27     
==========================================
+ Hits         7914     7941      +27     
  Misses       3863     3863              
Impacted Files Coverage Δ
src/trees/BoundingBox.h 100.00% <ø> (ø)
src/trees/MWNode.h 75.80% <ø> (-0.39%) :arrow_down:
src/utils/Plotter.cpp 0.00% <0.00%> (ø)
src/utils/periodic_utils.cpp 100.00% <ø> (ø)
src/trees/MWNode.cpp 76.93% <81.25%> (-0.10%) :arrow_down:
src/operators/OperatorState.h 100.00% <100.00%> (ø)
src/treebuilders/ABGVCalculator.cpp 87.20% <100.00%> (+0.30%) :arrow_up:
src/treebuilders/BSCalculator.cpp 100.00% <100.00%> (ø)
src/treebuilders/BandWidthAdaptor.h 100.00% <100.00%> (ø)
src/treebuilders/ConvolutionCalculator.cpp 98.35% <100.00%> (-0.01%) :arrow_down:
... and 21 more

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 504f03c...b2a26a7. Read the comment docs.

robertodr commented 3 years ago

Well, Minazo is useless right now. Does this affect the Python bindings in VAMPyR?

stigrj commented 3 years ago

Thanks for the feedback. Yes, I guess this will break vampyr...