-
Notifications
You must be signed in to change notification settings - Fork 346
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 FP16 for user buffer #690
Conversation
Signed-off-by: Hongbin Liu <[email protected]>
/te-ci pytorch |
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.
LGTM. Have you tested that userbuffers works as expected with FP16 in your use-case?
Signed-off-by: Kirthi Shankar Sivamani <[email protected]>
/te-ci pytorch |
Signed-off-by: Kirthi Shankar Sivamani <[email protected]>
/te-ci pytorch |
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 good
@timmoon10 A China customer, Baidu, verified the fp16+user buffer by hard-coding the dtype to fp16. |
* add_dtype_for_userbuf Signed-off-by: Hongbin Liu <[email protected]> * Update transformer_engine/pytorch/module/base.py Signed-off-by: Tim Moon <[email protected]> * Fix syntax Signed-off-by: Kirthi Shankar Sivamani <[email protected]> * Fix lint Signed-off-by: Kirthi Shankar Sivamani <[email protected]> --------- Signed-off-by: Hongbin Liu <[email protected]> Signed-off-by: Tim Moon <[email protected]> Signed-off-by: Kirthi Shankar Sivamani <[email protected]> Co-authored-by: Hongbin Liu <[email protected]> Co-authored-by: Tim Moon <[email protected]> Co-authored-by: Kirthi Shankar Sivamani <[email protected]> Signed-off-by: rachitg <[email protected]>
* add_dtype_for_userbuf Signed-off-by: Hongbin Liu <[email protected]> * Update transformer_engine/pytorch/module/base.py Signed-off-by: Tim Moon <[email protected]> * Fix syntax Signed-off-by: Kirthi Shankar Sivamani <[email protected]> * Fix lint Signed-off-by: Kirthi Shankar Sivamani <[email protected]> --------- Signed-off-by: Hongbin Liu <[email protected]> Signed-off-by: Tim Moon <[email protected]> Signed-off-by: Kirthi Shankar Sivamani <[email protected]> Co-authored-by: Hongbin Liu <[email protected]> Co-authored-by: Tim Moon <[email protected]> Co-authored-by: Kirthi Shankar Sivamani <[email protected]> Signed-off-by: kitefang <[email protected]>
* add_dtype_for_userbuf Signed-off-by: Hongbin Liu <[email protected]> * Update transformer_engine/pytorch/module/base.py Signed-off-by: Tim Moon <[email protected]> * Fix syntax Signed-off-by: Kirthi Shankar Sivamani <[email protected]> * Fix lint Signed-off-by: Kirthi Shankar Sivamani <[email protected]> --------- Signed-off-by: Hongbin Liu <[email protected]> Signed-off-by: Tim Moon <[email protected]> Signed-off-by: Kirthi Shankar Sivamani <[email protected]> Co-authored-by: Hongbin Liu <[email protected]> Co-authored-by: Tim Moon <[email protected]> Co-authored-by: Kirthi Shankar Sivamani <[email protected]>
No description provided.