-
Notifications
You must be signed in to change notification settings - Fork 26.3k
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
[doc] bf16/tf32 guide #14579
[doc] bf16/tf32 guide #14579
Conversation
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.
Looks great, just one fix is needed to go with the new doc front (image links are now different).
Co-authored-by: Sylvain Gugger <35901082+sgugger@users.noreply.github.com>
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.
@stas00 Looks good. Minor things. I am sorry for adding everything as comments, the GitHub "Insert a suggestion" button doesn't appear for me.
Thanks a lot for the proofreading and the corrections, @manuelciosici You couldn't make suggestions since the PR was merged already. I applied your corrections to this ongoing PR #14580 that collects small fixes. I even figured out how to make the right attribution to your suggestions ;)
|
* [doc] bf16/tf32 guide * expand * expand * Update docs/source/performance.md Co-authored-by: Sylvain Gugger <35901082+sgugger@users.noreply.github.com> Co-authored-by: Sylvain Gugger <35901082+sgugger@users.noreply.github.com>
This PR adds performance-specific docs as a follow up to #13207
--fp16_full_eval
--bf16
and--bf16_full_eval
tf32
This is the first pass, surely will add more down the road.
@sgugger