-
Notifications
You must be signed in to change notification settings - Fork 25.7k
DOC: update CrossEntropyLoss with note and example of incorrect target specification #155649
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/155649
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 89f38af with merge base 2259dbe ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@svekars any update on this? Thanks! |
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.
Thanks
|
Thank you both! |
|
@svekars any blockers on this PR getting merged? Thanks! 🚀 |
|
@svekars @mikaylagawarecki are there any blockers I can take care of to get this PR merged? |
|
@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 |
|
@svekars @mikaylagawarecki trying to sync with latest main to remove merge failures |
|
Pull workflow has not been scheduled for the PR yet. It could be because author doesn't have permissions to run those or skip-checks keywords were added to PR/commits, aborting merge. Please get/give approval for the workflows and/or remove skip ci decorators before next merge attempt. If you think this is a mistake, please contact PyTorch Dev Infra. |
|
@pytorchbot merge |
|
@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: trunk / macos-py3-arm64 / build Details for Dev Infra teamRaised by workflow job |
|
@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: trunk / macos-py3-arm64 / test (default, 1, 3, macos-m1-stable) Details for Dev Infra teamRaised by workflow job |
|
@mikaylagawarecki @svekars is it possible to rerun the one failing test to get this green? I'm not sure how to do that. I don't see any documentation on a pytorchbot command for that. Thank you for your help. Sorry a simple doc string fix is taking this much time and effort! |
|
@loganthomas I keep getting an error that disappears when I run it. Will run it again and send it your way to see if you can make heads or tails of it. It has something to do with the example in the docstring |
|
@loganthomas see error below: |
@mikaylagawarecki this looks like a precision issue maybe something that isnt such a big issue? |
|
@sekyondaMeta thank you! Yes, this appears to be a minor precision discrepancy and not blocking in the context of the example. For the main purpose of the example, these should be considered equivalent. |
|
Yea, I think you can either update the value or skip the doctest for that example by doing something like pytorch/torch/utils/cpp_extension.py Line 1175 in 20bdabb
|
|
@sekyondaMeta @mikaylagawarecki thank you both. is there a way to kick off a new test run? happy to do this is there is a build int command. |
|
we have to kick it off for you, but I just did that :) |
|
@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 |
|
@sekyondaMeta @mikaylagawarecki @svekars 🎉 thank you so much for all your help!!! |
…t specification (pytorch#155649) Fixes pytorch#134771 Pull Request resolved: pytorch#155649 Approved by: https://github.com/mikaylagawarecki Co-authored-by: Svetlana Karslioglu <svekars@meta.com> Co-authored-by: mikaylagawarecki <mikaylagawarecki@gmail.com>
…t specification (pytorch#155649) Fixes pytorch#134771 Pull Request resolved: pytorch#155649 Approved by: https://github.com/mikaylagawarecki Co-authored-by: Svetlana Karslioglu <svekars@meta.com> Co-authored-by: mikaylagawarecki <mikaylagawarecki@gmail.com>
Fixes #134771
cc @svekars @sekyondaMeta @AlannaBurke