-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[Cherry-pick] Collective communication APIs #46922
[Cherry-pick] Collective communication APIs #46922
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
d39ce87
to
868b1e1
Compare
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 单测时间设置
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.
for testcase, Please use np.testing.assert_allclose
in future PRs.
see: https://github.com/PaddlePaddle/community/blob/master/pfcc/call-for-contributions/code_style_improvement_for_unittest.md
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.
not reviewed this PR thoroughly.
given a +1 to help it pass CI check, after offline discussion.
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, this is a set of cherry-pick prs for 2.4, all comments are modified in origin pr of develop branch before cherry-pick here.
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
PR types
New features
PR changes
APIs
Describe
Cherry-pick from the following PRs, including bf16 dtype support for collective ops, new stream APIs and some bug fixes:
Related docs: PaddlePaddle/docs#5237