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

[XPU]. Fixed the mode error in pad3d. #9506

Merged
merged 2 commits into from
Nov 9, 2022
Merged

Conversation

wbn03
Copy link
Contributor

@wbn03 wbn03 commented Sep 27, 2022

修复了xpu上pad3d算子所有模式都使用constant的错误。

@paddle-bot
Copy link

paddle-bot bot commented Sep 27, 2022

Thanks for your contribution!

Copy link
Contributor

@wangleilei001 wangleilei001 left a comment

Choose a reason for hiding this comment

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

see the details

padding[2] = pads[2];
padding[3] = pads[3];
padding[4] = pads[0];
padding[5] = pads[1];
Copy link
Contributor

Choose a reason for hiding this comment

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

pads可能是一个元素吗?

in_dims[4],
in_dims[1],
in_dims[2],
in_dims[3],
Copy link
Contributor

Choose a reason for hiding this comment

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

建议使用前面的 n c d h w这些变量作为参数 ---- 理论上不应该存在is_ncdhw为两种值时调用两次api

@wbn03 wbn03 force-pushed the pad3d branch 2 times, most recently from b30e2ea to d2aebe8 Compare November 4, 2022 07:37
@wangleilei001
Copy link
Contributor

LGTM

Copy link
Collaborator

@zhupengyang zhupengyang left a comment

Choose a reason for hiding this comment

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

LGTM

@zhupengyang zhupengyang merged commit b7f5753 into PaddlePaddle:develop Nov 9, 2022
@wbn03 wbn03 deleted the pad3d branch November 16, 2022 09:09
newway pushed a commit to newway/Paddle-Lite that referenced this pull request Nov 19, 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.

None yet

4 participants