-
Notifications
You must be signed in to change notification settings - Fork 631
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
fix(lidar_apollo_instance_segmentation): fix critical bug #8444
fix(lidar_apollo_instance_segmentation): fix critical bug #8444
Conversation
Signed-off-by: kminoda <[email protected]>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
🙇 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8444 +/- ##
==========================================
- Coverage 23.89% 23.83% -0.07%
==========================================
Files 1380 1379 -1
Lines 101864 101423 -441
Branches 38783 38532 -251
==========================================
- Hits 24343 24174 -169
+ Misses 75075 74789 -286
- Partials 2446 2460 +14
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -47,7 +47,7 @@ | |||
<arg name="input/pointcloud" default="/sensing/lidar/concatenated/pointcloud" description="The topic will be used in the detection module"/> | |||
<arg name="mode" default="camera_lidar_fusion" description="options: `camera_lidar_radar_fusion`, `camera_lidar_fusion`, `lidar_radar_fusion`, `lidar` or `radar`"/> | |||
<arg name="data_path" default="$(env HOME)/autoware_data" description="packages data and artifacts directory path"/> | |||
<arg name="lidar_detection_model" default="centerpoint" description="options: `transfusion`, `centerpoint`, `apollo`, `pointpainting`, `clustering`"/> | |||
<arg name="lidar_detection_model" default="apollo" description="options: `transfusion`, `centerpoint`, `apollo`, `pointpainting`, `clustering`"/> |
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.
Is it for debug or want to change default to apollo?
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.
Thanks, that's a mistake. Fixed with the latest version.
87377f2
to
aba7e3c
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
CI keeps failing 😢 |
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
- I confirmed that this PR content is good to merge.
- I checked the CIs passed except
build-and-test-differential / clang-tidy-differential (pull_request)
- I checked the CIs passed except
- I did NOT check this PR by running autoware.
…undation#8444) Signed-off-by: kminoda <[email protected]> Co-authored-by: Shintaro Tomie <[email protected]>
Description
Original code included a critical bug which caused an empty output from apollo node.
Related links
Parent Issue:
How was this PR tested?
We've tested with
Note that this doesn't fix the issues we had internally: https://evaluation.tier4.jp/evaluation/reports/2921e69d-65b2-5067-af4b-8fffa4166d74?project_id=prd_jt
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.