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

Move group and all reduce from collective to communication #45848

Merged
merged 2 commits into from
Oct 10, 2022

Conversation

LiYuRio
Copy link
Contributor

@LiYuRio LiYuRio commented Sep 7, 2022

PR types

Others

PR changes

Others

Describe

Move group and all_reduce from collective.py to communication. Also this pr support using paddle.distributed.all_reduce in static mode.

@paddle-bot
Copy link

paddle-bot bot commented Sep 7, 2022

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@LiYuRio LiYuRio force-pushed the dev_remove_group branch 4 times, most recently from 71f84b1 to 46c13ef Compare September 22, 2022 03:35
@LiYuRio LiYuRio changed the title test Move group and all reduce from collective to communication Sep 22, 2022
@LiYuRio LiYuRio force-pushed the dev_remove_group branch 2 times, most recently from d46a82c to 7bd73c2 Compare September 22, 2022 14:34
@LiYuRio LiYuRio force-pushed the dev_remove_group branch 7 times, most recently from f427414 to 4c7afc2 Compare September 29, 2022 01:50
Copy link
Contributor

@gongweibao gongweibao left a comment

Choose a reason for hiding this comment

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

LGTM

@sljlp sljlp merged commit a0dffd3 into PaddlePaddle:develop Oct 10, 2022
XiaoguangHu01 pushed a commit that referenced this pull request Oct 17, 2022
* Support both use_calc_stream and sync_op in send recv APIs (#46023)

* Support both use_calc_stream and sync_op in allgather API (#46295)

* Support both use_calc_stream and sync_op in collective communication API (#46761)

* Move group and all reduce from collective to communication (#45848)

* Completes bfloat16 dtype for collective api in eager mode (#45844)

* Fix collective APIs cannot be recognized when building docs (#46962)

Co-authored-by: LiYuRio <63526175+LiYuRio@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants