Skip to content

Duplicating communicator in MPI backend#35

Merged
amartinhuertas merged 2 commits intogridap_distributedfrom
comm_dup
Oct 20, 2021
Merged

Duplicating communicator in MPI backend#35
amartinhuertas merged 2 commits intogridap_distributedfrom
comm_dup

Conversation

@fverdugo
Copy link
Copy Markdown
Collaborator

No description provided.

@fverdugo
Copy link
Copy Markdown
Collaborator Author

Fixes issue #33

Copy link
Copy Markdown
Collaborator

@amartinhuertas amartinhuertas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I now noticed that there are failing tests. The code looks good.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 20, 2021

Codecov Report

Merging #35 (9a89715) into gridap_distributed (36ad1f8) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                  Coverage Diff                   @@
##           gridap_distributed      #35      +/-   ##
======================================================
- Coverage               92.17%   92.12%   -0.05%     
======================================================
  Files                       8        8              
  Lines                    2057     2058       +1     
======================================================
  Hits                     1896     1896              
- Misses                    161      162       +1     
Impacted Files Coverage Δ
src/MPIBackend.jl 98.18% <100.00%> (-0.60%) ⬇️

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 36ad1f8...9a89715. Read the comment docs.

@fverdugo
Copy link
Copy Markdown
Collaborator Author

Now the tests are working!

@amartinhuertas amartinhuertas merged commit 13035ef into gridap_distributed Oct 20, 2021
@fverdugo fverdugo deleted the comm_dup branch October 20, 2021 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants