-
Notifications
You must be signed in to change notification settings - Fork 26.3k
Add test tracking operators without decompositions #90887
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
Conversation
This test inspects `torch.ops.aten` directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/90887
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 FailuresAs of commit fa6bdac: NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This test inspects `torch.ops.aten` directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
lezcano
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't find linalg_solve_triangular in the expected list, but that one has explicit CPU and CUDA dispatches. What's going on there?
This test inspects the dispatcher directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
This test inspects the dispatcher directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
ezyang
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems... fine, I guess? Automatically updating the expect file works right? Cc @SherlockNoMad
This test inspects the dispatcher directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
This test inspects the dispatcher directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
This test inspects the dispatcher directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
This test inspects the dispatcher directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
This test inspects the dispatcher directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
|
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
|
@pytorchbot revert -m 'Sorry for reverting your PR. We reverted #70988 in https://hud.pytorch.org/pytorch/pytorch/commit/acdd462b1a070790799ce4623ce8ecc83e197e81 and this test starts to fail. There is probably a dependency between the twos' -c weird |
|
@pytorchbot successfully started a revert job. Check the current status here. |
|
@peterbell10 your PR has been successfully reverted. |
This reverts commit 2740daf. Reverted #90887 on behalf of https://github.com/huydhn due to Sorry for reverting your PR. We reverted #70988 in https://hud.pytorch.org/pytorch/pytorch/commit/acdd462b1a070790799ce4623ce8ecc83e197e81 and this test starts to fail. There is probably a dependency between the twos
|
Please do a rebase to master and rerun the tests |
This test inspects `torch.ops.aten` directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. ghstack-source-id: 7728948 Pull Request resolved: pytorch#90887
This test inspects the dispatcher directly, so captures operators without `OpInfo` including internal helper operators and backward operators that might appear in a trace. [ghstack-poisoned]
|
@pytorchbot merge -f "Dynamo failure is pre-existing on master" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Stack from ghstack (oldest at bottom):
This test inspects the dispatcher directly, so captures operators without
OpInfoincluding internal helper operators and backward operators that mightappear in a trace.