Skip to content
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

Merged

Conversation

esteve
Copy link
Contributor

@esteve esteve commented Mar 22, 2023

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.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@github-actions github-actions bot added the component:perception Advanced sensor data processing and environment understanding. (auto-assigned) label Mar 22, 2023
@codecov
Copy link

codecov bot commented Mar 22, 2023

Codecov Report

Patch coverage has no change and project coverage change: -2.91% ⚠️

Comparison is base (c8f826d) 15.89% compared to head (ac32230) 12.99%.
Report is 1 commits behind head on main.

❗ Current head ac32230 differs from pull request most recent head 486802d. Consider uploading reports for the commit 486802d to get more accurate results

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     
Flag Coverage Δ *Carryforward flag
differential 0.00% <ø> (?)
total 13.07% <ø> (-2.82%) ⬇️ Carriedforward from 1ed2648

*This pull request uses carry forward flags. Click here to find out more.

see 1092 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@esteve esteve changed the title build(tensorrt_yolo): add DOWNLOAD_ARTIFACTS logic build(tensorrt_yolo): remove download logic from CMake Mar 24, 2023
@esteve esteve force-pushed the add-download-artifacts-tensorrt-yolo branch 2 times, most recently from 455f607 to ac32230 Compare March 28, 2023 11:52
@stale
Copy link

stale bot commented May 27, 2023

This pull request has been automatically marked as stale because it has not had recent activity.

@stale stale bot added the status:stale Inactive or outdated issues. (auto-assigned) label May 27, 2023
@stale stale bot removed the status:stale Inactive or outdated issues. (auto-assigned) label Jun 20, 2023
@esteve esteve force-pushed the add-download-artifacts-tensorrt-yolo branch from ac32230 to f7d55c9 Compare July 12, 2023 10:21
@esteve esteve marked this pull request as ready for review July 12, 2023 10:21
@esteve esteve requested a review from wep21 as a code owner July 12, 2023 10:21
@esteve
Copy link
Contributor Author

esteve commented Jul 12, 2023

@xmfcx @mitsudome-r this PR is now ready for review

Copy link
Contributor

@wep21 wep21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@esteve esteve force-pushed the add-download-artifacts-tensorrt-yolo branch from f7d55c9 to 486802d Compare September 6, 2023 09:45
@esteve esteve marked this pull request as draft September 8, 2023 14:59
@esteve esteve marked this pull request as ready for review September 20, 2023 10:05
@esteve esteve added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Sep 20, 2023
@esteve esteve enabled auto-merge (squash) September 20, 2023 10:05
@esteve esteve merged commit a5feaba into autowarefoundation:main Sep 20, 2023
@esteve esteve deleted the add-download-artifacts-tensorrt-yolo branch September 20, 2023 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:perception Advanced sensor data processing and environment understanding. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

2 participants