Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1533: Fix typo in MPI access guard error message #1686

Merged
merged 1 commit into from
Mar 15, 2022
Merged

Conversation

PhilMiller
Copy link
Member

Fixes #1533

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (gcc-6, ubuntu, mpich)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (gcc-5, ubuntu, mpich)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (clang-3.9, ubuntu, mpich)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (clang-5.0, ubuntu, mpich)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (gcc-9, ubuntu, mpich, zoltan)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (gcc-8, ubuntu, mpich, address sanitizer)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (nvidia cuda 10.1, ubuntu, mpich)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (gcc-10, ubuntu, openmpi, no LB)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (clang-9, ubuntu, mpich)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (clang-10, alpine, mpich)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (nvidia cuda 11.0, ubuntu, mpich)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (clang-10, ubuntu, mpich)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@codecov
Copy link

codecov bot commented Feb 23, 2022

Codecov Report

Merging #1686 (487e500) into develop (bdda95f) will increase coverage by 0.03%.
The diff coverage is 0.00%.

❗ Current head 487e500 differs from pull request most recent head cd8863f. Consider uploading reports for the commit cd8863f to get more accurate results

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1686      +/-   ##
===========================================
+ Coverage    83.77%   83.80%   +0.03%     
===========================================
  Files          773      772       -1     
  Lines        26871    26861      -10     
===========================================
  Hits         22510    22510              
+ Misses        4361     4351      -10     
Impacted Files Coverage Δ
tests/unit/runtime/test_mpi_access_guards.cc 88.88% <0.00%> (ø)
src/vt/scheduler/priority_manip.cc 50.00% <0.00%> (-5.56%) ⬇️
src/vt/pipe/pipe_manager.cc 55.55% <0.00%> (-4.45%) ⬇️
src/vt/trace/trace_event.cc 75.00% <0.00%> (-1.20%) ⬇️
src/vt/utils/compress/compressor.cc 83.33% <0.00%> (-0.88%) ⬇️
.../collection/balance/model/proposed_reassignment.cc 80.00% <0.00%> (-0.77%) ⬇️
...vt/vrt/collection/balance/temperedlb/temperedlb.cc 70.26% <0.00%> (-0.22%) ⬇️
src/vt/context/context.cc 100.00% <0.00%> (ø)
src/vt/group/group_info.cc 100.00% <0.00%> (ø)
src/vt/sequence/seq_closure.cc 100.00% <0.00%> (ø)
... and 15 more

@github-actions
Copy link

github-actions bot commented Feb 23, 2022

PR tests (gcc-7, ubuntu, mpich, trace runtime, LB)

Build for cd8863f

Compilation - successful

Testing - passed

Build log

@PhilMiller PhilMiller force-pushed the 1533-typo branch 2 times, most recently from 7387470 to 487e500 Compare March 15, 2022 17:12
@PhilMiller PhilMiller merged commit 6316827 into develop Mar 15, 2022
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.

Typo in MPI access guard message
2 participants