Skip to content

Conversation

@BujSet
Copy link
Contributor

@BujSet BujSet commented Jul 29, 2025

Summary

Minor refactor to standardize how operator names are printed when debugging. Especially useful when hitting a missing operator error.

@pytorch-bot
Copy link

pytorch-bot bot commented Jul 29, 2025

🔗 Helpful Links

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

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

❌ 6 New Failures, 45 Unrelated Failures

As of commit 101f7b3 with merge base b1d09fd (image):

NEW FAILURES - The following jobs have failed:

BROKEN TRUNK - The following jobs 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 29, 2025
@BujSet BujSet requested a review from lucylq July 29, 2025 00:07
@BujSet BujSet self-assigned this Jul 29, 2025
@BujSet BujSet changed the title Started refactoring operators to use op_name format Refactoring Portable Operators to Standardize op_name Format Jul 29, 2025
@github-actions
Copy link

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@BujSet
Copy link
Contributor Author

BujSet commented Jul 29, 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 29, 2025
@BujSet BujSet added ciflow/trunk ciflow/binaries ciflow/binaries/all Release PRs with this label will build wheels for all python versions and removed release notes: none Do not include this in the release notes labels Jul 29, 2025
@BujSet BujSet force-pushed the refactor_op_names branch from 50678b3 to 018cd1b Compare July 30, 2025 00:44
@BujSet BujSet force-pushed the refactor_op_names branch from 018cd1b to 98f2017 Compare August 14, 2025 18:26
@BujSet BujSet marked this pull request as ready for review August 14, 2025 18:29
Copy link
Contributor

@lucylq lucylq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the refactoring work!

@BujSet BujSet merged commit cad1214 into pytorch:main Aug 14, 2025
200 of 251 checks passed
@BujSet BujSet deleted the refactor_op_names branch August 14, 2025 20:37
agrima1304 pushed a commit to agrima1304/executorch that referenced this pull request Aug 26, 2025
…#12941)

### Summary
Minor refactor to standardize how operator names are printed when
debugging. Especially useful when hitting a missing operator error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries/all Release PRs with this label will build wheels for all python versions ciflow/binaries ciflow/trunk 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.

3 participants