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

Benchmarks: Add Benchmark - Add ib validation tool source code #191

Merged
merged 15 commits into from Oct 21, 2021

Conversation

yukirora
Copy link
Contributor

@yukirora yukirora commented Sep 9, 2021

Description
Add IB validation tool source code. IB validation tool is a tool to validate IB traffic of different pattern in multi nodes flexibly

Major Revision

  • Add ib validation tool source code
  • Add cmake file to build the source code

@yukirora yukirora requested a review from a team as a code owner September 9, 2021 04:36
@codecov
Copy link

codecov bot commented Sep 9, 2021

Codecov Report

Merging #191 (e74d022) into main (a030d71) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #191   +/-   ##
=======================================
  Coverage   88.56%   88.56%           
=======================================
  Files          59       59           
  Lines        2922     2922           
=======================================
  Hits         2588     2588           
  Misses        334      334           
Flag Coverage Δ
cpu-unit-test 74.48% <ø> (ø)
cuda-unit-test 88.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 a030d71...e74d022. Read the comment docs.

@cp5555 cp5555 self-assigned this Sep 28, 2021
int size = run_pairs_in_parallel.size();
// invoke function to run cmd in multi threads mode for each rank in the pairs
unordered_map<int, std::future<float>> threads;
for (int index = 0; index < size; index++) {

Choose a reason for hiding this comment

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

should we limit max number of active threads? (given that some clusters can be really large, and also with multiple NICs per node)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

add limit for the max num of threads at the same time, if expired, split the line

@microsoft microsoft deleted a comment from abuccts Oct 13, 2021
@cp5555 cp5555 added benchmarks SuperBench Benchmarks micro-benchmarks Micro Benchmark Test for SuperBench Benchmarks labels Oct 20, 2021
@yukirora yukirora enabled auto-merge (squash) October 21, 2021 02:58
@yukirora yukirora merged commit 2664850 into main Oct 21, 2021
@yukirora yukirora deleted the v-yujiang/ib-mpi branch October 21, 2021 03:14
@cp5555 cp5555 mentioned this pull request Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
benchmarks SuperBench Benchmarks micro-benchmarks Micro Benchmark Test for SuperBench Benchmarks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants