Skip to content

[ET-VK] Renaming slice_channel and slice_batch_height_width files to reflect change in its utility. #9170

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

Merged
merged 2 commits into from
Mar 11, 2025

Conversation

pytorchbot
Copy link
Collaborator

This PR was created by the merge bot to help merge the original PR into the main branch.
ghstack PR number: #9137 by @trivedivivek
^ Please use this as the source of truth for the PR details, comments, and reviews
ghstack PR base: https://github.com/pytorch/executorch/tree/gh/trivedivivek/60/base
ghstack PR head: https://github.com/pytorch/executorch/tree/gh/trivedivivek/60/head
Merge bot PR base: https://github.com/pytorch/executorch/tree/gh/trivedivivek/59/orig
Merge bot PR head: https://github.com/pytorch/executorch/tree/gh/trivedivivek/60/orig
@diff-train-skip-merge

…as slice dim.

Pull Request resolved: #9136

This diff is a code change for the Slice op to shorten its code when packed dimension is not the same as the slice dimension.
ghstack-source-id: 271070837
@exported-using-ghexport

Differential Revision: [D70737264](https://our.internmc.facebook.com/intern/diff/D70737264/)
…reflect change in its utility.

Pull Request resolved: #9137

This diff renames files `slice_batch_height_width.y*` to `slice_unpacked_dim.*` and `slice_channel.*` to `slice_packed_dim.*` to reflect the change in their utility after enabling all dim packing for slice op.
ghstack-source-id: 271070838
@exported-using-ghexport

Differential Revision: [D70919994](https://our.internmc.facebook.com/intern/diff/D70919994/)
@pytorchbot pytorchbot requested a review from SS-JIA as a code owner March 11, 2025 23:01
Copy link

pytorch-bot bot commented Mar 11, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/9170

Note: Links to docs will display an error until the docs builds have been completed.

⏳ No Failures, 8 Pending

As of commit 41bf83a with merge base e763a83 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 11, 2025
Base automatically changed from gh/trivedivivek/59/orig to main March 11, 2025 23:34
@SS-JIA SS-JIA merged commit 1462960 into main Mar 11, 2025
48 checks passed
@SS-JIA SS-JIA deleted the gh/trivedivivek/60/orig branch March 11, 2025 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants