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

[oneDNN][cherry-pick] Fix to #33282 , added support of X input broadcasting to one… #33845

Merged
merged 1 commit into from
Jul 9, 2021

Conversation

jczaja
Copy link
Contributor

@jczaja jczaja commented Jun 29, 2021

PR types

Bug fixes

PR changes

OPs

Describe

This is a backport of fix to #33282.

…sting to oneDNN elementwise ops (PaddlePaddle#33549)

* - fix to PaddlePaddle#33282

* - Increased threshold for elementwise_mul_bf16 grad

* -disabled faulty UT

* - fix to approval
@paddle-bot-old
Copy link

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

@jczaja jczaja added the Intel label Jun 29, 2021
@jakpiase jakpiase self-requested a review June 30, 2021 12:16
Copy link
Contributor

@jakpiase jakpiase left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@lidanqing-intel lidanqing-intel left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks !

@jczaja
Copy link
Contributor Author

jczaja commented Jul 1, 2021

@wzzju , @juncaipeng Please merge (it is a cherry-pick from develop).

Copy link
Contributor

@juncaipeng juncaipeng left a comment

Choose a reason for hiding this comment

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

LGTM

@paddle-bot-old
Copy link

paddle-bot-old bot commented Jul 7, 2021

Sorry to inform you that 640bb19's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

Copy link
Contributor

@Superjomn Superjomn left a comment

Choose a reason for hiding this comment

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

LGTM

@Superjomn Superjomn merged commit f2f2fd8 into PaddlePaddle:release/2.1 Jul 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants