-
Notifications
You must be signed in to change notification settings - Fork 100
Add collective benchmark and correctness check #814
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
base: main
Are you sure you want to change the base?
Changes from 12 commits
4474359
cdb0383
0605022
b302796
2fe6b1e
f1a5a7d
44dab3b
dc37dd6
569acc3
ab567ef
ad97f72
c8a49fa
493e3b3
ce03bae
f830639
fac9467
a861ccc
efb6865
04c995f
cbfe3c3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,18 @@ | ||
| # Copyright (c) Microsoft Corporation. | ||
| # Licensed under the MIT License. | ||
|
|
||
| from .mscclpp_op import MscclppAllReduce1, MscclppAllReduce2, MscclppAllReduce3, MscclppAllReduce4, MscclppAllReduce5 | ||
| __all__ = [ | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should we change the naming for allreduce1, allreduce2, etc. and make it more descriptive?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Will deprotect these algo as we move most of the algo implementation to collective folder. Will address in following pr. |
||
| "MscclppAllReduce1", | ||
| "MscclppAllReduce2", | ||
| "MscclppAllReduce3", | ||
| "MscclppAllReduce4", | ||
| "MscclppAllReduce5", | ||
| ] | ||
|
|
||
|
|
||
| def __getattr__(name): | ||
| if name in __all__: | ||
| from . import mscclpp_op | ||
|
|
||
| return getattr(mscclpp_op, name) | ||
| raise AttributeError(f"module {__name__!r} has no attribute {name!r}") | ||
Uh oh!
There was an error while loading. Please reload this page.