-
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
Fix TRT destroying a runtime before destroying deserialized engines #53937
Fix TRT destroying a runtime before destroying deserialized engines #53937
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
❌ The PR is not created using PR's template. You can refer to this Demo. |
@tianshuo78520a, |
PR-CI-GpuPS-PSLIB 的 log 顯示
這 PR 也並沒有改動 protocol buffer. 問題似乎是 CI 設置有問題? |
这条任务并不是Required,因此不会影响PR正常Merge。 |
已经豁免 |
PR-CI-Windows 的 log 顯示 [2827/3874] Building CUDA object paddle\fluid\operators\CMakeFiles\paddle_operators_unity.dir\paddle_operators_unity_5_cu.cu.obj
FAILED: paddle/fluid/operators/CMakeFiles/paddle_operators_unity.dir/paddle_operators_unity_5_cu.cu.obj
C:\Python37\Scripts\sccache.exe C:\PROGRA~1\NVIDIA~2\CUDA\v10.2\bin\nvcc.exe ... paddle_operators_unity_5_cu.cu
../paddle/fluid/framework/eigen.h(159): error C2065: 'NumIndices': undeclared identifier 錯誤是發生在 eigen.h,也和這 PR 無關 |
windows可能是cache原因导致的,我正在尝试重跑CI |
錯誤仍然是 eigen
|
已经通过,CheckPRTemplate可以通过修改PR模板进行重新触发操作。 |
@yuanlehome , @qingqing01 , and @zhangjun , |
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
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
Bug fixes
PR changes
Others
Description
Currently TRT-related UTs will report the following error:
The error is self-explained: Destroying a runtime before destroying deserialized engines created by the runtime is not allowed. This PR fixes this issue.