Skip to content

Enable optimized op_le broadcast against 1 element tensor #9507

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 1 commit into from
Mar 22, 2025

Conversation

manuelcandales
Copy link
Contributor

Summary: Will enable full broadcasting support in a future PR. But right now, enabling the 1-element case, to resolve #9433

Differential Revision: D71652593

Copy link

pytorch-bot bot commented Mar 21, 2025

🔗 Helpful Links

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

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

❌ 4 New Failures

As of commit cea5201 with merge base d980ce0 (image):

NEW FAILURES - The following jobs have failed:

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 21, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71652593

@manuelcandales manuelcandales added the release notes: ops & kernels Changes to the opset and any new / changed kernel implementations label Mar 21, 2025
@guangy10
Copy link
Contributor

Cool! I have verified the fix. Thanks @manuelcandales ! Can you please fix the linter so we can merge it and then cherry-pick to the release branch. It's affecting several HF models, so would want this fix to be included in the 0.6 release. huggingface/optimum-executorch#14

Summary:

Will enable full broadcasting support in a future PR. But right now, enabling the 1-element case, to resolve  pytorch#9433

Reviewed By: larryliu0820, guangy10

Differential Revision: D71652593
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71652593

@facebook-github-bot facebook-github-bot merged commit 78f0e67 into pytorch:main Mar 22, 2025
77 of 82 checks passed
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. fb-exported release notes: ops & kernels Changes to the opset and any new / changed kernel implementations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

KernelCall failed at instruction 0:16 in operator aten::le.Tensor_out: 0x12
4 participants