-
Notifications
You must be signed in to change notification settings - Fork 676
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(traffic_light_fine_detector): remove download logic from CMake #4896
build(traffic_light_fine_detector): remove download logic from CMake #4896
Conversation
Signed-off-by: Alexey Panferov <[email protected]>
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #4896 +/- ##
==========================================
- Coverage 15.81% 15.12% -0.69%
==========================================
Files 1581 1576 -5
Lines 109896 107499 -2397
Branches 34411 32769 -1642
==========================================
- Hits 17377 16259 -1118
- Misses 73393 73458 +65
+ Partials 19126 17782 -1344
*This pull request uses carry forward flags. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Signed-off-by: Alexey Panferov <[email protected]>
…ight_fine_detector' into remove_downloading_traffic_light_fine_detector
…c_light_fine_detector
…c_light_fine_detector
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.
@lexavtanke thanks
…c_light_fine_detector
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
Description
See #3137
According to discussion above remove downloading logic from building stage. For now It will be possible to download model files either by ansible or manually as explained in this PR
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.