Skip to content

Conversation

@betterpig
Copy link
Contributor

@betterpig betterpig commented Aug 11, 2021

PR types

New features

PR changes

OPs

Describe

Support npu kernel for pad3d op.
Note: cann op PadV3 only support mode="constant" and constant_value=0 now (click for more info) , so only realized pad3d_grad of constant mode. And add error raise and test cases accordingly. Besides, cann op PadV3 not support double and int64 data type.

log info is as follows:
Pasted Graphic 11
Pasted Graphic 3

test result :
image

image

@paddle-bot-old
Copy link

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

zhwesky2010
zhwesky2010 previously approved these changes Aug 12, 2021
Copy link
Contributor

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

LGTM

qili93
qili93 previously approved these changes Aug 20, 2021
Copy link
Contributor

@qili93 qili93 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

@qili93 qili93 left a comment

Choose a reason for hiding this comment

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

LGTM

@qili93 qili93 merged commit ef517a5 into PaddlePaddle:develop Aug 20, 2021
@betterpig betterpig deleted the pad3d_op_npu branch August 20, 2021 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants