search

jipolanco / PencilArrays.jl

Distributed Julia arrays using the MPI protocol
https://jipolanco.github.io/PencilArrays.jl/dev/
MIT License
60 stars 8 forks source link

Add support for reduction of multiple `PencilArray`s #46

Closed corentin-dev closed 2 years ago

corentin-dev commented 2 years ago

Small change to be able to do:

rr₂ = mapreduce((x,y)->real(x*conj(y)),+,r,r₂)

where r and r₂ are PencilArray.

I guess there are much cleaner ways to do it...

codecov-commenter commented 2 years ago

Codecov Report

Merging #46 (ca6b067) into master (aa24546) will increase coverage by 0.01%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   96.62%   96.64%   +0.01%     
==========================================
  Files          21       21              
  Lines        1156     1162       +6     
==========================================
+ Hits         1117     1123       +6     
  Misses         39       39
Impacted Files Coverage Δ
src/reductions.jl 100.00% <100.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 aa24546...ca6b067. Read the comment docs.

corentin-dev commented 2 years ago

This is way cleaner, thank you!