-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
add adadelta op for xpu, test=kunlun #47661
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
@@ -105,6 +106,8 @@ XPUOpMap& get_kl2_ops() { | |||
XPUKernelSet({pOpKernelType(vartype::FP32, XPUPlace()), | |||
pOpKernelType(vartype::FP16, XPUPlace())})}, | |||
{"clip", XPUKernelSet({pOpKernelType(vartype::FP32, XPUPlace())})}, | |||
{"clip_by_norm", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个的单测呢?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
单测之前就有了,不过没有按照目前的格式进行修改。那我修改下重新提交吧。
def test_check_output(self): | ||
self.check_output() | ||
|
||
class TestAdadeltaOp2(OpTest): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里是XPUOpTest吧?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
攒着下次一起改,节省机器资源
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
目前单测可以跑过,下次pr的时候再修改一下。
你的PR已合入Paddle库,请关注后续测试结果。 |
PR types
New features
PR changes
OPs
Describe
1、add adadelta op for xpu
2、xpu2_op_list.h增加clip_by_norm声明