-
Notifications
You must be signed in to change notification settings - Fork 21.7k
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
[DTensor] Remove compute_local_offset from _utils.py and replace current usage with new API #108965
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/108965
Note: Links to docs will display an error until the docs builds have been completed. ✅ You can merge normally! (3 Unrelated Failures)As of commit 393b9c6 with merge base 3b265e0 ( BROKEN TRUNK - The following jobs failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
UNSTABLE - The following job failed but was likely due to flakiness present on trunk and has been marked as unstable:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
This pull request was exported from Phabricator. Differential Revision: D49130050 |
This pull request was exported from Phabricator. Differential Revision: D49130050 |
…ent usage with new API (pytorch#108965) Summary: This diff: - Remove compute_local_offset from _utils.py - Replace current usage with new API compute_local_shape_and_global_offset. More context of why we introduced the new API: pytorch#108547 Test Plan: CI Differential Revision: D49130050
b06a4a0
to
393b9c6
Compare
This pull request was exported from Phabricator. Differential Revision: D49130050 |
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 thanks for cleaning this up.
Also please make sure all CI passed. |
@pytorchmergebot merge -i |
Merge failedReason: This PR has internal changes and must be landed via Phabricator Details for Dev Infra teamRaised by workflow job |
Closing and replacing it with #109096 |
Separating internal changes with OSS changes. This PR contains removing the compute_local_offset from the OSS directory only. This replaces #108965 Pull Request resolved: #109096 Approved by: https://github.com/wanchaol, https://github.com/fduwjj
Summary:
This diff:
More context of why we introduced the new API:
#108547
Test Plan: CI
Differential Revision: D49130050