-
Notifications
You must be signed in to change notification settings - Fork 688
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
logPartitionInfo: fix to list all LogicalDeviceIDs for the partition #5999
base: master
Are you sure you want to change the base?
Conversation
This pull request was exported from Phabricator. Differential Revision: D39456245 |
This pull request was exported from Phabricator. Differential Revision: D39456245 |
…ytorch#5999) Summary: Pull Request resolved: pytorch#5999 PartitionerUtils.cpp: function 'logPartitionInfo' lists only the first LogicalDeviceIDs for the partition. Reviewed By: caogao Differential Revision: D39456245 fbshipit-source-id: 1a2dee170cddd475d247105838cc9cadf2a93c6f
a75a15d
to
ff676a2
Compare
…ytorch#5999) Summary: Pull Request resolved: pytorch#5999 PartitionerUtils.cpp: function 'logPartitionInfo' lists only the first LogicalDeviceIDs for the partition. Reviewed By: caogao Differential Revision: D39456245 fbshipit-source-id: 89783d87e9d8022d04c7ef12bee23a73fdd20a8f
ff676a2
to
eb3216d
Compare
This pull request was exported from Phabricator. Differential Revision: D39456245 |
Reviewed By: caogao Differential Revision: D39456245 fbshipit-source-id: a9ea5b1975090d52520a38736220ad97a9d09bb2
eb3216d
to
4c2310a
Compare
This pull request was exported from Phabricator. Differential Revision: D39456245 |
Hi @andreykareev! Thank you for your pull request. We require contributors to sign our Contributor License Agreement, and yours needs attention. You currently have a record in our system, but the CLA is no longer valid, and will need to be resubmitted. ProcessIn order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with If you have received this in error or have any questions, please contact us at cla@meta.com. Thanks! |
Summary: PartitionerUtils.cpp: function 'logPartitionInfo' lists only the first LogicalDeviceIDs for the partition.
Reviewed By: caogao
Differential Revision: D39456245