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
[BE][PG NCCL] Improve input mismatch error msg #107281
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/107281
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 6a62073 with merge base d0e50d9 (): This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D48363238 |
" with world_size", | ||
world_size, | ||
" and ", | ||
num_devices, |
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.
this is always 1 afaik, so can also exclude it.
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! Double check the nit.
"but got ", | ||
tensor_lists[i].size(), | ||
" inputs", | ||
" with world_size", |
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.
nit: I think?
" with world_size", | |
" with world_size ", |
This pull request was exported from Phabricator. Differential Revision: D48363238 |
Summary: Pull Request resolved: pytorch#107281 Test Plan: CI Reviewed By: daniellepintz, H-Huang, awgu Differential Revision: D48363238 fbshipit-source-id: e2b0573dd02e4225820cd90f2c7feaa42c90136a
a17af5f
to
cef1ca4
Compare
Summary: Pull Request resolved: pytorch#107281 Test Plan: CI Reviewed By: daniellepintz, H-Huang, awgu Differential Revision: D48363238 fbshipit-source-id: 705bc6cc63d4173fba630f69471fbfa0d35bb636
This pull request was exported from Phabricator. Differential Revision: D48363238 |
cef1ca4
to
6a62073
Compare
@pytorchbot merge -f "CI done" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Test Plan: CI Differential Revision: D48363238 Pull Request resolved: pytorch#107281 Approved by: https://github.com/awgu, https://github.com/H-Huang, https://github.com/fegin
Test Plan: CI
Differential Revision: D48363238