-
Notifications
You must be signed in to change notification settings - Fork 312
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
[ET-VK][BE][ez] vTensor cleanup 7/N - Blanket replacement of packed_dim_whcn_idx
with packed_dim
#5484
Conversation
…dim_whcn_idx` with `packed_dim` ## Context `packed_dim_whcn_idx` is a bit too verbose. Replace it with `packed_dim` for brevity. Differential Revision: [D63032323](https://our.internmc.facebook.com/intern/diff/D63032323/) [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/5484
Note: Links to docs will display an error until the docs builds have been completed. ⏳ No Failures, 1 PendingAs of commit 20f68b5 with merge base 8ef6c79 (): This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D63032323 |
…dim_whcn_idx` with `packed_dim` ## Context `packed_dim_whcn_idx` is a bit too verbose. Replace it with `packed_dim` for brevity. Differential Revision: [D63032323](https://our.internmc.facebook.com/intern/diff/D63032323/) ghstack-source-id: 243540489 Pull Request resolved: #5484
…of `packed_dim_whcn_idx` with `packed_dim`" ## Context `packed_dim_whcn_idx` is a bit too verbose. Replace it with `packed_dim` for brevity. Differential Revision: [D63032323](https://our.internmc.facebook.com/intern/diff/D63032323/) [ghstack-poisoned]
…dim_whcn_idx` with `packed_dim` Pull Request resolved: #5484 ## Context `packed_dim_whcn_idx` is a bit too verbose. Replace it with `packed_dim` for brevity. Differential Revision: [D63032323](https://our.internmc.facebook.com/intern/diff/D63032323/) ghstack-source-id: 243563524
This pull request was exported from Phabricator. Differential Revision: D63032323 |
This pull request has been merged in 90d5191. |
Stack from ghstack (oldest at bottom):
packed_dim_whcn_idx
withpacked_dim
#5484gpu_memory_layout
as a source of truth, usepacked_dim_whcn_idx
directly #5479Context
packed_dim_whcn_idx
is a bit too verbose. Replace it withpacked_dim
for brevity.Differential Revision: D63032323