[CI] remove TensorRT from PaddleTest#3241
Conversation
ShigureNyako
left a comment
There was a problem hiding this comment.
结论:当前 head 068e7ff97b2e704a85c91759c3d98f9ce71b1155 仍需修改。
本轮有两处阻塞项:删除 Serving 的禁用态兼容参数会破坏普通 CPU/GPU 用例;删除混合参数化的 OCR 测试时一并丢失了普通 GPU 预测覆盖。细节见行级评论。
聚焦静态验证通过:77 个变更 Python 文件 py_compile、37 个变更 Shell 文件 bash -n、git diff --check;404 个变更 YAML 用 PyYAML safe_load 为 401 个通过、3 个失败,3 个在基线同样失败。当前 GitHub 仅有 license/cla 成功,没有执行测试结果可覆盖上述回归。
| args.max_body_size = 512 * 1024 * 1024 | ||
| args.use_encryption_model = False | ||
| args.use_multilang = False | ||
| args.use_trt = False |
There was a problem hiding this comment.
这里不能删除 args.use_trt:它不是 TRT case,而是调用未同步修改的 paddle_serving_server.serve.start_gpu_card_model 所需的兼容参数。当前 test_serve.py:96,105,133,163 都先取 default_args() 再调用该函数;Serving 实现仍会直接读取 args.use_trt。删掉后普通 CPU/GPU Serving 用例会在启动前因 AttributeError 失败。请保留这个 False 默认值;它只表示不启用 TRT。
There was a problem hiding this comment.
复查当前 head 6e2a8f183ee7afc6ee80b9d47049f1908ea0c29d:本次提交仅修改 models/AutomaticTestSystem/test_ocr_acc.py,这里的 default_args() 仍未恢复 args.use_trt = False,因此该兼容性 blocker 仍然存在。
There was a problem hiding this comment.
已复查当前 head 00d233c5d532314866b2ab1291fb75cb822469ed:本次提交原样恢复 args.use_trt = False;实际加载 default_args() 验证该属性存在且为 False。Serving 兼容性问题已解决。
There was a problem hiding this comment.
在这里修不是最终解决方法,Serving 也应该做 trt 退场,不过那是后续的事了
|
|
||
| @allure.story("predict") | ||
| @pytest.mark.parametrize("yml_name", get_model_list()) | ||
| @pytest.mark.parametrize("use_tensorrt", [True, False]) |
There was a problem hiding this comment.
这个参数化测试不全是 TRT 覆盖:use_tensorrt=False 分支会标记为 _GPU,并在原 test_ocr_acc.py:202 以 use_gpu=True, use_tensorrt=False 执行普通 GPU 预测。当前 head 只剩 test_ocr_acc.py:177 的 use_gpu=False CPU/MKLDNN 调用,AutomaticTestSystem 中已没有普通 OCR GPU 预测入口。请保留或拆出 GPU-only 分支,只删除 use_tensorrt=True 的覆盖。
There was a problem hiding this comment.
已复查当前 head 6e2a8f183ee7afc6ee80b9d47049f1908ea0c29d:新增 test_ocr_accuracy_predict_gpu,并以 test_ocr_rec_predict(True, False) 恢复普通 GPU、TRT 关闭态的预测覆盖;该文件 py_compile 与本次提交 git diff --check 均通过。此项已解决。
ShigureNyako
left a comment
There was a problem hiding this comment.
结论:当前 head 6e2a8f183ee7afc6ee80b9d47049f1908ea0c29d 仍需修改。
本次提交仅修改 models/AutomaticTestSystem/test_ocr_acc.py:新增 GPU-only 用例并调用 test_ocr_rec_predict(True, False),已恢复普通 GPU、TRT 关闭态的预测覆盖;该项已在原线程确认解决。
Serving blocker 未处理:inference/serving_api_test/paddle_serving_server/util.py 仍缺少 args.use_trt = False,而未同步修改的 start_gpu_card_model 仍直接读取该参数。已在原线程补充当前 head 证据,不重复新增行级评论。
本次增量验证:新增 Python 文件改动通过 py_compile,068e7ff..6e2a8f1 通过 git diff --check。当前 GitHub 仍只有 license/cla 成功,没有执行测试结果。
ShigureNyako
left a comment
There was a problem hiding this comment.
已复查当前 head 00d233c5d532314866b2ab1291fb75cb822469ed,此前两个 blocker 均已解决:
- Serving fixture 已恢复
args.use_trt = False;实际调用default_args()验证该兼容属性存在且保持禁用。 - OCR GPU-only 用例仍以
test_ocr_rec_predict(True, False)保留普通 GPU、TRT 关闭态覆盖。
本次增量仅恢复一行 Serving 默认参数,无新范围。全量静态复核通过:76 个变更 Python 文件 py_compile、37 个变更 Shell 文件 bash -n、git diff --check;404 个变更 YAML 为 401 个通过、3 个与基线相同的既有失败。当前 GitHub 仅有 license/cla 成功,没有执行型 CI;仓库未配置 required checks,当前 merge state 为 clean。
代码审查结论:批准。
关联 PR
Paddle PR #79444 将 TensorRT 从默认 CI 构建和必测链路中退场,并明确把 PaddleTest 中 TRT 能力的清理留给本仓库处理。本 PR 不再局限于删除失败 case 或停止调度,而是从 PaddleTest 中完整删除 TRT 专用的测试、执行和配置能力。
变更内容
清理结果
trt子串,仅来自已发布的共享 Docker 镜像、wheel/预测库制品名称,以及PETRTransformer标识;这些内容不构成 PaddleTest 的 TRT 功能入口或配置能力。验证
py_compile。bash -n。develop基线中已存在不可解析状态,本 PR 未新增 YAML 解析回归。git diff --check通过。license/cla已通过。未运行依赖 GPU 的全量端到端测试;本 PR 的目标是删除 PaddleTest 中的 TRT 测试及执行能力,不修改 Paddle 仓库内的 TensorRT 产品实现。