-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
remove the tricks for paddle.fluid.layers.ops.func
#33731
remove the tricks for paddle.fluid.layers.ops.func
#33731
Conversation
Thanks for your contribution! |
fedd804
to
faa6c6f
Compare
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.
LGTM
@@ -30,7 +30,6 @@ | |||
import pkgutil | |||
import logging | |||
import argparse | |||
import paddle |
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.
这个import删除并放到下面的函数中,是为了运行test_print_signatures.py
测试用例,可以在不安装paddle的环境上执行此测试用例。
2. remove the tricks for paddle.fluid.layers.ops.func
ab417ca
to
d024d6d
Compare
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.
LGTM
output of running python3 tools/print_signatures.py paddle
may needs a clear format.
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.
LGTM
PR types
Others
PR changes
Others
Describe
paddle.fluid.layers.ops.func
Gpu samplecode test On PR-CPU-Py2 #33634