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
Support randn_like() for NT #96528
Support randn_like() for NT #96528
Conversation
[ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/96528
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 2b4d3c2: This comment was automatically generated by Dr. CI and updates every 15 minutes. |
@@ -499,13 +499,15 @@ def test_zero_(self): | |||
t.fill_(0.) | |||
self.assertEqual(nt_ub, t) | |||
|
|||
@parametrize("func", [torch.ones_like, torch.zeros_like], |
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.
Note: This currently passes, but I can imagine an alternative implementation of randn_like()
for NT that doesn't have the property of "generating all random numbers at once -> gives same numbers as iteratively generating numbers matching underlying component sizes", and it could be argued this is no less correct. So I'm open to better forms of testing.
@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 |
Merge failedReason: 1 jobs have failed, first few of them are: linux-binary-manywheel / manywheel-py3_8-cuda11_7-test / test Details for Dev Infra teamRaised by workflow job |
@pytorchbot merge -f "spurious failures" |
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 |
Merge failedReason: Command
Details for Dev Infra teamRaised by workflow job |
To satisfy an internal ask. [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 |
To satisfy an internal ask. Pull Request resolved: #96528 Approved by: https://github.com/mikaylagawarecki, https://github.com/cpuhrsch ghstack-source-id: d6e453446e8923ae94187a587601681c70450e7e
To satisfy an internal ask. Pull Request resolved: pytorch/pytorch#96528 Approved by: https://github.com/mikaylagawarecki, https://github.com/cpuhrsch
To satisfy an internal ask. Pull Request resolved: pytorch/pytorch#96528 Approved by: https://github.com/mikaylagawarecki, https://github.com/cpuhrsch
Stack from ghstack (oldest at bottom):
To satisfy an internal ask.