Skip to content

[Paddle TensorRT] add pd_op.max converter #67956

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

Merged
merged 11 commits into from
Sep 6, 2024

Conversation

Layssy
Copy link
Contributor

@Layssy Layssy commented Sep 3, 2024

PR Category

Inference

PR Types

New features

Description

card-71500
添加max op 的 marker 和 converter

Copy link

paddle-bot bot commented Sep 3, 2024

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@@ -0,0 +1,58 @@
# Copyright (c) 2024 PaddlePaddle Authors. All Rights Reserved.
Copy link
Contributor

Choose a reason for hiding this comment

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

有了test_converter_max就不需要有这个文件了,删除

@Layssy Layssy changed the title [Paddle TensorRT] add new converter [Paddle TensorRT] add pd_op.max converter Sep 3, 2024
@@ -0,0 +1,41 @@
# Copyright (c) 2024 PaddlePaddle Authors. All Rights Reserved.
Copy link
Contributor

Choose a reason for hiding this comment

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

CMakeLists.txt加一个限制时间的属性,否则就超时

@@ -0,0 +1,41 @@
# Copyright (c) 2024 PaddlePaddle Authors. All Rights Reserved.
#
Copy link
Contributor

Choose a reason for hiding this comment

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

该文件建议重命名为test_converter_math.py,后续所有math.py下的converter单测都会放到这里

@paddle-bot paddle-bot bot added the contributor External developers label Sep 3, 2024
@@ -14,4 +14,5 @@ if(NOT WIN32 AND TENSORRT_FOUND)
set_tests_properties(test_converter_conv2d PROPERTIES TIMEOUT "100")
set_tests_properties(test_converter_add PROPERTIES TIMEOUT "100")
set_tests_properties(test_converter_batch_norm PROPERTIES TIMEOUT "100")
set_tests_properties(test_converter_max PROPERTIES TIMEOUT "100")
Copy link
Contributor

Choose a reason for hiding this comment

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

你不是叫test_converter_math吗

@YuanRisheng YuanRisheng merged commit 95052cc into PaddlePaddle:develop Sep 6, 2024
29 checks passed
@luotao1
Copy link
Contributor

luotao1 commented Sep 10, 2024

hi, @Layssy

  • 非常感谢你对飞桨的贡献,我们正在运营一个PFCC组织,会通过定期分享技术知识与发布开发者主导任务的形式持续为飞桨做贡献,详情可见 https://github.com/luotao1 主页说明。
  • 如果你对PFCC有兴趣,请发送邮件至 ext_paddle_oss@baidu.com,我们会邀请你加入~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants