-
Notifications
You must be signed in to change notification settings - Fork 274
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
coll: enable topology-aware CVARs in json #6782
Conversation
4dd2e38
to
34309ec
Compare
test:mpich/ch4/gpu |
test:mpich/ch4/gpu |
test:mpich/ch4/gpu |
test:mpich/ch4/gpu |
test:mpich/ch4/gpu |
Note: we need make sure that tests failures are not related. This could be due to the Jenkins node issue, but we need resolve that first before passing this PR. |
I will try the tests on main to see if these errors appear. |
It seems like these test failures exist in the main branch. I created an empty PR and run test:mpich/ch4/gpu (#6960). The test failures in this PR can be found on that PR as well. |
test:mpich/ch4/gpu/ofi |
Refer to #6648 (comment) on test failures. |
Allow setting up topology-aware CVARs in collective tuning json file for bcast, ireduce and allreduce.
Add k value when caching the topo_aware and topo_aware_k trees. Add overhead, lat_diff_groups, lat_diff_switches and lat_same_switches when caching topo_wave tree.
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.
LGTM
Pull Request Description
Allow setting up topology-aware CVARs in collective tuning json file for bcast, reduce, and allreduce.
Author Checklist
Particularly focus on why, not what. Reference background, issues, test failures, xfail entries, etc.
Commits are self-contained and do not do two things at once.
Commit message is of the form:
module: short description
Commit message explains what's in the commit.
Whitespace checker. Warnings test. Additional tests via comments.
For non-Argonne authors, check contribution agreement.
If necessary, request an explicit comment from your companies PR approval manager.