-
Notifications
You must be signed in to change notification settings - Fork 9
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
1723 Include user-defined fields in json LB data files #1728
Conversation
PR tests (clang-5.0, ubuntu, mpich) Build for 7759537
|
PR tests (clang-3.9, ubuntu, mpich) Build for 7759537
|
PR tests (clang-9, ubuntu, mpich) Build for 7759537
|
PR tests (gcc-5, ubuntu, mpich) Build for 7759537
|
PR tests (gcc-10, ubuntu, openmpi, no LB) Build for 7759537
|
PR tests (gcc-9, ubuntu, mpich, zoltan) Build for 7759537
|
PR tests (gcc-6, ubuntu, mpich) Build for 7759537
|
PR tests (gcc-7, ubuntu, mpich, trace runtime, LB) Build for 7759537
|
Codecov Report
@@ Coverage Diff @@
## develop #1728 +/- ##
===========================================
- Coverage 84.65% 83.83% -0.82%
===========================================
Files 774 773 -1
Lines 27026 26965 -61
===========================================
- Hits 22879 22607 -272
- Misses 4147 4358 +211
|
PR tests (nvidia cuda 11.0, ubuntu, mpich) Build for 7759537
|
PR tests (nvidia cuda 10.1, ubuntu, mpich) Build for 7759537
|
PR tests (clang-10, alpine, mpich) Build for 7759537
|
PR tests (gcc-8, ubuntu, mpich, address sanitizer) Build for 7759537
|
PR tests (clang-10, ubuntu, mpich) Build for 7759537
|
Overall, looks great! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
7ccdfaa
to
6376c04
Compare
PR tests (clang-11, ubuntu, mpich) Build for 7759537
|
PR tests (intel icpx, ubuntu, mpich) Build for 7759537
|
6376c04
to
7759537
Compare
PR tests (clang-13, ubuntu, mpich) Build for 7759537
|
PR tests (clang-12, ubuntu, mpich) Build for 7759537
|
PR tests (clang-14, ubuntu, mpich) Build for 7759537
|
PR tests (gcc-11, ubuntu, mpich) Build for 7759537
|
PR tests (gcc-12, ubuntu, mpich) Build for 7759537
|
This PR adds the ability to dump items stored in collection elements as user-defined data in LB data json files. I also added a commit that removes some stale lines of code from an existing unit test.
Closes #1723