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.

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

Fix layer_norm accuracy #29434

Merged
merged 1 commit into from
Dec 7, 2020

Conversation

zhiqiu
Copy link
Contributor

@zhiqiu zhiqiu commented Dec 7, 2020

PR types

Bug fixes

PR changes

OPs

Describe

Fix accuracy problem introduced in #29169

@paddle-bot-old
Copy link

paddle-bot-old bot commented Dec 7, 2020

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

@zhiqiu zhiqiu requested a review from phlrain December 7, 2020 11:27
@zhiqiu zhiqiu merged commit a040c05 into PaddlePaddle:develop Dec 7, 2020
windstamp pushed a commit to windstamp/Paddle that referenced this pull request Jan 5, 2021
lanxianghit pushed a commit that referenced this pull request Jan 7, 2021
#29576) (#30110)

* Layer norm fp16 (#29169)

* add fp16 for layer_norm op

* revert layernorm api

* fix forward

* fix forward

* fix backward for layernorm with fp16

* fix unit test for layernorm with fp16

* fix with_mkldnn compile error for layernorm with fp16

* 1. revert to PADDLE_ENFORCE_NOT_NULL, 2. change static_cast<float> to static_cast<U>

* fix with_mkldnn compile error for layernorm with fp16

* fix with_mkldnn compile error for layernorm with fp16

Co-authored-by: zhiqiu <[email protected]>

* fix layer_norm accuracy (#29434)

* Layernorm opt (#29522)

* layernorm fw opt

* layernorm bw opt

* fix typo, test=develop

* remove const dim3 for windows CI compatibility

* merge develop

Co-authored-by: zlsh80826 <[email protected]>

* Fix compile problem when cuda_arch < 6000 (#29576)

* fix compile problem when cuda_arch < 6000

* refine code

* refine code

Co-authored-by: zhiqiu <[email protected]>
Co-authored-by: zlsh80826 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants