-
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
[Inference] add config.enable_low_precision_io api and remove rely on AnalysisConfig::Precison in trt #52485
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
86cd435
to
44f13a0
Compare
Hi @yuanlehome , is there any unit test for the new config keep_io_datatype? |
Currently not available, I need to resolve some unitest issues on CI first. |
A simple network fed into Paddle Inference with the following conditions is good to check the input datatype of TRT engine.
The result predictor in Paddle Inference should take an FP16 GPU tensor as input and an FP16 tensor as output. To test the IR graph that adds cast, a program consisting of some operators unable to be converted into TRT is also required. |
Good tips. Maybe I want to wait for this PR to merge before proceeding. |
Hi @yuanlehome , I would like to add more information for the UT: There are two parts of the UT
If there are any questions, please let me know. |
Sorry to inform you that 44f13a0's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
6d7bf24
to
13f08cb
Compare
c7e2b05
to
b0efe3e
Compare
…fig::Precision in trt
a8d4b10
to
f293ff8
Compare
a56156d
to
4ff9507
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 for set_tests_properties(test_trt_inference_fp16_io PROPERTIES TIMEOUT 300)
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
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
… AnalysisConfig::Precison in trt (PaddlePaddle#52485)
PR types
New features
PR changes
APIs
Description
fix bug: #54042, #54032, #54129