Skip to content

Fix a bug in native::_do_all_gather related to group#2947

Merged
vfdev-5 merged 7 commits intopytorch:masterfrom
sadra-barikbin:Feature-allgather-tensors-with-different-shapes-but-the-same-ndim
Jun 19, 2023
Merged

Fix a bug in native::_do_all_gather related to group#2947
vfdev-5 merged 7 commits intopytorch:masterfrom
sadra-barikbin:Feature-allgather-tensors-with-different-shapes-but-the-same-ndim

Conversation

@sadra-barikbin
Copy link
Copy Markdown
Collaborator

@sadra-barikbin sadra-barikbin commented May 17, 2023

-Fix a bug in native::_do_all_gather related to group
-Improve idist.all_gather docstring

Check list:

  • New tests are added (if a new feature is added)
  • New doc strings: description and/or example code are in RST format
  • Documentation is updated (if required)

@github-actions github-actions bot added the module: distributed Distributed module label May 17, 2023
@sadra-barikbin sadra-barikbin mentioned this pull request May 17, 2023
3 tasks
@sadra-barikbin sadra-barikbin force-pushed the Feature-allgather-tensors-with-different-shapes-but-the-same-ndim branch from d156ac3 to eeda68b Compare June 16, 2023 15:20
@vfdev-5
Copy link
Copy Markdown
Collaborator

vfdev-5 commented Jun 16, 2023

@sadra-barikbin please update PR title and description to match the content

@sadra-barikbin sadra-barikbin changed the title Feature allgather tensors with different shapes but the same ndim Fix a bug in native::_do_all_gather related to group Jun 16, 2023
Copy link
Copy Markdown
Collaborator

@vfdev-5 vfdev-5 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @sadra-barikbin !

@vfdev-5 vfdev-5 enabled auto-merge (squash) June 19, 2023 07:38
@vfdev-5 vfdev-5 merged commit ffed9f3 into pytorch:master Jun 19, 2023
@sadra-barikbin sadra-barikbin deleted the Feature-allgather-tensors-with-different-shapes-but-the-same-ndim branch June 19, 2023 08:30
@vfdev-5
Copy link
Copy Markdown
Collaborator

vfdev-5 commented Jun 20, 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: distributed Distributed module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants