Skip to content
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.

Sign up for GitHub

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

Jump to bottom

remove bf16 #38133

Merged
merged 4 commits into from
Dec 15, 2021
Merged

remove bf16 #38133

merged 4 commits into from
Dec 15, 2021

Conversation

b3602sss
Copy link
Contributor

@b3602sss b3602sss commented Dec 14, 2021 •
edited
Loading

PR types

Bug fixes

PR changes

OPs

Describe

quantize is FP32 only, so disable non-fp32 case.

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

zhouping828
zhouping828 reviewed Dec 15, 2021
View reviewed changes
Copy link

@zhouping828 zhouping828 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@b3602sss b3602sss merged commit 49108ef into PaddlePaddle:develop Dec 15, 2021
@b3602sss b3602sss deleted the quantize branch December 15, 2021 06:40
Caozhou1995 pushed a commit to Caozhou1995/Paddle that referenced this pull request Dec 29, 2021
@b3602sss @Caozhou1995
remove bf16 (PaddlePaddle#38133)
beca9a9 
* remove bf16

* remove comments

* remove wrong return

* fix UT
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhouping828 zhouping828 zhouping828 left review comments

@jiweibo jiweibo jiweibo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@b3602sss @jiweibo @zhouping828