-
Notifications
You must be signed in to change notification settings - Fork 25.7k
[BE] small typo+lint fixes for einsum/sumproduct_pair #85709
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
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/85709
Note: Links to docs will display an error until the docs builds have been completed. ✅ No Failures, 3 PendingAs of commit 722329b: This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
This PR has been accepted with the accept2ship label. Attempting to merge now. @pytorchbot merge -l |
|
@pytorchbot successfully started a merge job. Check the current status here and land check progress here. |
Merge failedReason: Failed to merge; some land checks failed: pull, pull / linux-bionic-cuda11.6-py3.10-gcc7 / test (functorch, 1, 1, linux.4xlarge.nvidia.gpu) If you believe this is an error, you can use the old behavior with Please reach out to the PyTorch DevX Team with feedback or questions! Details for Dev Infra teamRaised by workflow job |
|
@pytorchbot merge -g The land checks failure is flaky |
|
@pytorchbot successfully started a merge job. Check the current status here. |
|
Hey @janeyx99. |
Easy review! This PR fixes some typos + lints + clarifies some instructions Pull Request resolved: pytorch#85709 Approved by: https://github.com/soulitzer
Easy review! This PR fixes some typos + lints + clarifies some instructions Pull Request resolved: #85709 Approved by: https://github.com/soulitzer
Easy review! This PR fixes some typos + lints + clarifies some instructions