-
Notifications
You must be signed in to change notification settings - Fork 672
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
build(tensorrt_yolo): remove download logic from CMake #3142
build(tensorrt_yolo): remove download logic from CMake #3142
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #3142 +/- ##
==========================================
- Coverage 15.89% 12.99% -2.91%
==========================================
Files 1578 1271 -307
Lines 108871 89405 -19466
Branches 33591 26865 -6726
==========================================
- Hits 17303 11614 -5689
+ Misses 72989 65544 -7445
+ Partials 18579 12247 -6332
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
455f607
to
ac32230
Compare
This pull request has been automatically marked as stale because it has not had recent activity. |
ac32230
to
f7d55c9
Compare
@xmfcx @mitsudome-r this PR is now ready for review |
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
Signed-off-by: Esteve Fernandez <[email protected]>
f7d55c9
to
486802d
Compare
Description
See #3137
Depends on autowarefoundation/autoware_common#164
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.