-
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
[Paddle TensorRT No.2] Add pd_op.elu converter #69278
[Paddle TensorRT No.2] Add pd_op.elu converter #69278
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
标题把 No.2 放 [] 里边叭,不然不会自动统计,issue 里已经修改过了 |
收到 |
@@ -20,6 +20,21 @@ | |||
import paddle | |||
|
|||
|
|||
class TestEluTRTPattern(TensorRTBaseTest): |
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.
再加一个"int64"的单测,把np.float32改为"float32"
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.
if in_dynamic_or_pir_mode():
return _C_ops.elu(x, alpha)
else:
check_variable_and_dtype(
x, 'x', ['float16', 'uint16', 'float32', 'float64'], 'elu'
)
helper = LayerHelper("elu", **locals())
out = helper.create_variable_for_type_inference(x.dtype)
helper.append_op(
type='elu',
inputs={'X': x},
outputs={'Out': out},
attrs={'alpha': alpha},
)
return out
不支持int64
冲突了 |
@lizexu123 please review the pr if u are free. thx. |
self.check_trt_result() | ||
|
||
|
||
class TestEluTRTPatternCase2(TensorRTBaseTest): |
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.
再加一个self.enable_fp16=True的检查,如果过了就可以approve了
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.
这个可以先合入,后续再添加也行
冲突啦 |
Sorry to inform you that 1be8daa's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
@YuanRisheng conflict resloved. please review if u are free. thx. |
PR Category
User Experience
PR Types
New features
Description
新增了pd_op.elu Marker和Converter