-
Notifications
You must be signed in to change notification settings - Fork 25.7k
clarify that torch.finfo.tiny is the smallest normal number
#63241
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
💊 CI failures summary and remediationsAs of commit fd4013a (more details on the Dr. CI page):
ci.pytorch.org: 1 failedThis comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions to the (internal) Dr. CI Users group. |
|
@dagitses has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
…ber" This is a common source of confusion, but it matches the NumPy behavior. Fixes #44010 Fixes #59526 Differential Revision: [D30307646](https://our.internmc.facebook.com/intern/diff/D30307646) [ghstack-poisoned]
|
@dagitses has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
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.
Nice fix
thanks! |
…ber" This is a common source of confusion, but it matches the NumPy behavior. Fixes #44010 Fixes #59526 Differential Revision: [D30307646](https://our.internmc.facebook.com/intern/diff/D30307646) [ghstack-poisoned]
|
@dagitses has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Stack from ghstack:
torch.finfo.tinyis the smallest normal number #63241This is a common source of confusion, but it matches the NumPy
behavior.
Fixes #44010
Fixes #59526
Differential Revision: D30307646