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

[PHI] Migrate pool2d and pool2d_grad kernels #47423

Merged
merged 48 commits into from
Nov 4, 2022

Conversation

piotrekobi
Copy link
Contributor

PR types

Others

PR changes

Others

Describe

Migrates the following oneDNN operator fluid kernels to PHI:

  • pool2d
  • pool2d_grad

chenwhql and others added 30 commits September 20, 2022 11:20
@paddle-bot
Copy link

paddle-bot bot commented Oct 27, 2022

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added contributor External developers status: proposed labels Oct 27, 2022
@piotrekobi piotrekobi marked this pull request as draft October 27, 2022 15:35
@piotrekobi piotrekobi marked this pull request as ready for review November 2, 2022 09:55
YuanRisheng
YuanRisheng previously approved these changes Nov 3, 2022
Silv3S
Silv3S previously approved these changes Nov 3, 2022
Copy link
Member

@Silv3S Silv3S 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/fluid/operators/mkldnn/test_mkldnn_op_nhwc.cc Outdated Show resolved Hide resolved
Co-authored-by: Sławomir Siwek <slawomir.siwek@intel.com>
piotrekobi and others added 2 commits November 3, 2022 12:38
Co-authored-by: Sławomir Siwek <slawomir.siwek@intel.com>
Co-authored-by: Sławomir Siwek <slawomir.siwek@intel.com>
@YuanRisheng YuanRisheng merged commit ca4bed7 into PaddlePaddle:develop Nov 4, 2022
Copy link
Contributor

@sfraczek sfraczek left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers Intel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants