Skip to content

Conversation

@BujSet
Copy link
Contributor

@BujSet BujSet commented Jul 27, 2025

Summary

Operator and dtype selective build requires operator names to match operators in use. This changes corrects the name for mean operator. Change is similar to that in #12755.

@pytorch-bot
Copy link

pytorch-bot bot commented Jul 27, 2025

🔗 Helpful Links

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

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

✅ You can merge normally! (1 Unrelated Failure)

As of commit 909fb61 with merge base 00e3f99 (image):

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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 Jul 27, 2025
@BujSet
Copy link
Contributor Author

BujSet commented Jul 27, 2025

@pytorchbot label "release notes: none"

@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Jul 27, 2025
@BujSet BujSet marked this pull request as ready for review July 28, 2025 16:24
@BujSet BujSet requested a review from lucylq July 28, 2025 16:24
@lucylq
Copy link
Contributor

lucylq commented Jul 28, 2025

thanks for fixing these!

@BujSet BujSet merged commit de89397 into pytorch:main Jul 28, 2025
99 of 101 checks passed
@BujSet BujSet deleted the op_mean_name_fix branch July 28, 2025 16:26
@BujSet BujSet self-assigned this Jul 28, 2025
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. release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants