Skip to content

Conversation

@jakpiase
Copy link
Contributor

PR types

New features

PR changes

OPs

Describe

Added logsoftmax BF16/FP32 FWD oneDNN kernel with tests in both old and new Paddle's UT frameworks

@paddle-bot-old
Copy link

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

@jakpiase
Copy link
Contributor Author

@piotrekobiIntel please review this PR

Copy link
Contributor

@piotrekobi piotrekobi left a comment

Choose a reason for hiding this comment

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

LGTM

@jakpiase jakpiase requested a review from jczaja February 23, 2022 18:24
Copy link
Contributor

@jczaja jczaja left a comment

Choose a reason for hiding this comment

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

LGTM.

@jczaja
Copy link
Contributor

jczaja commented Feb 24, 2022

@baoachun Please start your review

Copy link
Contributor

@baoachun baoachun left a comment

Choose a reason for hiding this comment

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

LGTM

@jczaja jczaja merged commit 584844e into PaddlePaddle:develop Feb 25, 2022
liutiexing pushed a commit to liutiexing/Paddle that referenced this pull request Feb 28, 2022
liutiexing pushed a commit to liutiexing/Paddle that referenced this pull request Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants