-
Notifications
You must be signed in to change notification settings - Fork 801
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
Add height of cluster with the most points method #323
Add height of cluster with the most points method #323
Conversation
Signed-off-by: kosuke55 <[email protected]>
17e3311
to
ae30a70
Compare
ae30a70
to
657cc05
Compare
@pfankhauser |
ping @maximilianwulf @SteveMacenski @Marwan99 |
Hey, could you retarget it to the main branch? |
@maximilianwulf |
Yes. Are you working enterily with ROS2? |
Yes. I'm working enterily with ROS2. |
Could you in any case create a 2nd MR onto master? |
OK, will you plan to review PR for master before this PR? |
@maximilianwulf |
friendly ping @maximilianwulf @SteveMacenski @Marwan99 |
Thank you. |
Thank you for the work.
I added the method to use the mean value of the cluster with the most points as elevation value.
This method may be able to estimate more ground-like values.
0: Smallest value among the average values of each cluster (current method)
1: Mean value of the cluster with the most points (new method)
(height_thresh is a parameter for this methods to reduce the influence of point clouds that exist in high locations such as ceilings.)