- Notifications
You must be signed in to change notification settings - Fork26.3k
Fix is_nonzero for more than one elem tensors#149637
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
[ghstack-poisoned]
pytorch-botbot commentedMar 20, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
🔗 Helpful Links🧪 See artifacts and rendered test results athud.pytorch.org/pr/149637
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 New Failure, 2 Unrelated FailuresAs of commit297884d with merge base2c4bc65 ( NEW FAILURE - The following job has failed:
FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
tugsbayasgalan commentedMar 20, 2025
@tugsbayasgalan has imported this pull request. If you are a Meta employee, you can view this diffon Phabricator. |
cc ezyang SherlockNoMad EikanWang jgong5 wenzhe-nrvDifferential Revision: [D71560442](https://our.internmc.facebook.com/intern/diff/D71560442)[ghstack-poisoned]
tugsbayasgalan commentedMar 20, 2025
@tugsbayasgalan has imported this pull request. If you are a Meta employee, you can view this diffon Phabricator. |
cc ezyang SherlockNoMad EikanWang jgong5 wenzhe-nrvDifferential Revision: [D71560442](https://our.internmc.facebook.com/intern/diff/D71560442)[ghstack-poisoned]
Pull Requestresolved:#149637cc@ezyang@SherlockNoMad@EikanWang@jgong5@wenzhe-nrv@imported-using-ghimportDifferential Revision: [D71560442](https://our.internmc.facebook.com/intern/diff/D71560442/)ghstack-source-id: 273256663
facebook-github-bot commentedMar 21, 2025
This pull request wasexported from Phabricator. Differential Revision:D71560442 |
facebook-github-bot commentedMar 22, 2025
@pytorchbot merge -i (Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally) |
pytorchmergebot commentedMar 22, 2025
Merge startedYour change will be merged while ignoring the following 3 checks:pull / linux-docs / build-docs-python-false,trunk / linux-focal-rocm6.3-py3.10 / test (default, 1, 2, linux.rocm.gpu.2),inductor / linux-jammy-cpu-py3.9-gcc11-inductor / test (dynamic_cpu_inductor_huggingface, 1, 1, linux.8xlarge.amx) Learn more about merging in thewiki. Questions? Feedback? Please reach out to thePyTorch DevX Team |
Differential Revision: [D71560442](https://our.internmc.facebook.com/intern/diff/D71560442)Pull Requestresolved:pytorch#149637Approved by:https://github.com/pianpwk
…154164)This was added in#149637,torch._check can handle unbacked there is no need for size oblivious reasoning here.Note this does not make is_nonzero unbacked friendly. but that is a different story.I ran the test added in#149637 for veirfication.Pull Requestresolved:#154164Approved by:https://github.com/aorenste,https://github.com/bobrenjc93ghstack dependencies:#154154
Uh oh!
There was an error while loading.Please reload this page.
Stack fromghstack (oldest at bottom):
cc@ezyang@SherlockNoMad@EikanWang@jgong5@wenzhe-nrv
Differential Revision:D71560442