-
Notifications
You must be signed in to change notification settings - Fork 667
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
feat(trtexec_vendor): add vendor package for trtexec #1821
Conversation
b509460
to
11cfd53
Compare
If you plan to release this package, I think it's better to manage this package in a separate repository and add it to |
11cfd53
to
e466bec
Compare
3939184
to
17db227
Compare
@kenji-miyake I won't make this package released because ros build farm cannot handle cuda libraries. |
17db227
to
532f70f
Compare
Codecov ReportBase: 10.45% // Head: 10.44% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1821 +/- ##
==========================================
- Coverage 10.45% 10.44% -0.02%
==========================================
Files 1231 1221 -10
Lines 87879 87744 -135
Branches 20501 20498 -3
==========================================
- Hits 9191 9164 -27
+ Misses 69175 69106 -69
+ Partials 9513 9474 -39
*This pull request uses carry forward flags. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
I have checked that almost the same performance is reproduced using engine files built with the following trtexec command.
But, it takes more than 3 times than the lidar_centerpoint package implementation to build two TensorRT engine files. |
Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp>
Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com>
Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com>
Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp>
Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp>
294cd19
to
a2e8969
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.
I have checked that trtexec can convert onnx files and lidar_centerpoint can load engine files.
…ion#1821) * feat(trtexec_vendor): add vendor package for trtexec Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * support the previous CUDA environemt Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * remove required for nvidia libraries Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * add sampleUtils.cpp if trt version is greater than 8.4 Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * do not install trt submodules Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com>
…ion#1821) * feat(trtexec_vendor): add vendor package for trtexec Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * support the previous CUDA environemt Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * remove required for nvidia libraries Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * add sampleUtils.cpp if trt version is greater than 8.4 Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * do not install trt submodules Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com>
…ion#1821) * feat(trtexec_vendor): add vendor package for trtexec Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * support the previous CUDA environemt Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * remove required for nvidia libraries Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * add sampleUtils.cpp if trt version is greater than 8.4 Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * do not install trt submodules Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com>
…ion#1821) * feat(trtexec_vendor): add vendor package for trtexec Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * support the previous CUDA environemt Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * remove required for nvidia libraries Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * add sampleUtils.cpp if trt version is greater than 8.4 Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * do not install trt submodules Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com>
…ion#1821) * feat(trtexec_vendor): add vendor package for trtexec Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * support the previous CUDA environemt Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * remove required for nvidia libraries Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com> * add sampleUtils.cpp if trt version is greater than 8.4 Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> * do not install trt submodules Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp> Co-authored-by: Yusuke Muramatsu <yukke42@users.noreply.github.com>
Description
a vendor package for
trtexec
Related links
Tests performed
Please check if you can build tensorrt engine from onnx file with
trtexec
Notes for reviewers
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.