-
Notifications
You must be signed in to change notification settings - Fork 704
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
obstacle pointcloud based validator node Incorrectly removes the small DetectedObjects (traffic cone) from centerpoint. #6938
Comments
@sgNicola Thanks for your contribution. Could you share example data to reproduce this problem. |
Hi, I have shared the ros2bag in this Google Drive link [provided.] |
|
Hi, @meliketanrikulu , I have checked the visiting property, could you please try again? Thank you. |
Hi @sgNicola |
Thank you for your attention, I have updated the link [
https://drive.google.com/drive/folders/1CtjPF1jNsm2bRsyisEOX7EJDEtoroJnX?usp=sharing].
Could you please check again?
Amadeusz Szymko ***@***.***> 于2024年7月8日周一 14:57写道:
… Hi @sgNicola <https://github.com/sgNicola>
Did you fix the issue already? If not, I would like to reproduce it but
your link does not have rosbag file, I see only the meta file.
—
Reply to this email directly, view it on GitHub
<#6938 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A3UKP4EV4ZGDCQAW7WTRPB3ZLIZ7HAVCNFSM6AAAAABHMLBCGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJTGE4DMNZUGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@sgNicola
I suggest to move on ROS 2 humble if it's possible and tune the config file, but keep that in mind you will end up with more false positives. I hope it will help you a lot! |
Thank you for your help!
Amadeusz Szymko ***@***.***> 于2024年7月10日周三 16:43写道:
… @sgNicola <https://github.com/sgNicola>
I reproduced your issue and here is brief summary:
1. Apart of fact the cone was detected as pedestrian, indeed
obstacle_pointcloud_based_validator removes tiny objects.
2. This issue was addressed in past PR #2894
<#2894>.
Unfortunately for galactic branch users, this feature is not included.
3. Beside this dynamic number of points calculation, this detection is
still being removed. I believe our default config gives us trade off
between maximizing false positives / minimizing true positives removal.
Users may play with settings to fit their scenarios. Based on distance and
number of points (5~ in your example) I quickly calculated desired values -
for your case you would need to change desired class min_points_num to
5 and min_points_and_distance_ratio to 100 here
<https://github.com/autowarefoundation/autoware_launch/blob/main/autoware_launch/config/perception/object_recognition/detection/detected_object_validation/obstacle_pointcloud_based_validator.param.yaml>
.
obstacle_pointcloud_based_validator.png (view on web)
<https://github.com/autowarefoundation/autoware.universe/assets/37396312/4e0e2d84-9e2d-4cc7-a8fa-a9ddc1da3c9b>
I suggest to move on ROS 2 humble if it's possible and tune the config
file, but keep that in mind you will end up with more false positives. I
hope it will help you a lot!
—
Reply to this email directly, view it on GitHub
<#6938 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A3UKP4AJB2DY6BN5YHJ5TNLZLTX3PAVCNFSM6AAAAABHMLBCGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJZHEYTKOJSGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Closing as resolved 👍🏻 |
Checklist
Description
The obstacle pointcloud-based validator node is incorrectly removing small DetectedObjects, such as traffic cones, identified by the centerpoint node. This leads to a misdetection.


Expected behavior
The obstacle pointcloud-based validator node should accurately validate and retain all DetectedObjects recognized by the centerpoint node, including small objects like traffic cones.
Actual behavior
The CenterPoint node detects small objects (e.g., traffic cones) with low frequency, but the obstacle point cloud-based validator node subsequently removes these detections.
Steps to reproduce
Versions
-OS: Ubuntu20.04
-ROS2: ROS2 galactic
-Autoware: Autoware.universe galactic
Possible causes
Additional context
No response
The text was updated successfully, but these errors were encountered: