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

Added bilinear and nearest interp v2 oneDNN FP32 kernels #32312

Merged
merged 12 commits into from
Apr 21, 2021

Conversation

jakpiase
Copy link
Contributor

PR types

New features

PR changes

OPs

Describe

Added oneDNN FP32 kernels for nearest_interp_v2 and bilinear_interp_v2

@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

@lidanqing-intel @jczaja @wozna Could you please review this PR?

auto scale_data = get_new_data_from_tensor<float>(scale_tensor);
std::copy(scale_data.begin(), scale_data.end(), scales.begin());
while (scale_data.size() < 3) scales.push_back(scale_data[0]);
} else {
Copy link
Contributor

@lidanqing-intel lidanqing-intel Apr 19, 2021

Choose a reason for hiding this comment

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

Hi, Is this else branch only part that is different from interpolate_mkldnn_v1?
Maybe function std::is_same for type comparison could be used for this case and we could combine interpolate v1 and v2?

@@ -0,0 +1,210 @@
# Copyright (c) 2018 PaddlePaddle Authors. All Rights Reserved.
Copy link
Contributor

Choose a reason for hiding this comment

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

year 2021

@lidanqing-intel
Copy link
Contributor

@jczaja @wozna Hi, if you have time, could you please also review this PR? We need 1 more review I guess

@lidanqing-intel
Copy link
Contributor

LGTM

jczaja
jczaja previously approved these changes Apr 20, 2021
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

@luotao1 luotao1 merged commit 5d19f8d into PaddlePaddle:develop Apr 21, 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.

4 participants