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

the sensor height change? #26

Closed
Nothand0212 opened this issue May 25, 2023 · 1 comment
Closed

the sensor height change? #26

Nothand0212 opened this issue May 25, 2023 · 1 comment

Comments

@Nothand0212
Copy link

I set the sensor height to 3.0,but on some case it will change like below?

the data is collected on boat. In some case, it work well; but some case, patchwork++ can not master.

did I need to change other params?

[ RNR ] Num of noises : 57
Flushed patches
[ CZM ] Divides pointcloud into the concentric zone model
=========== Temporal Ground Revert (TGR) ===========
[1, 8] mean_flatness: 0.0106572, stdev_flatness: 0.00265974
8th flat_sector_candidate / flatness: 0.0195439 / line_variable: 5.9849 / ground_num : 137
FINAL REJECT
====================================================
sensor height: 0.709483
elevation_thr_  :   -0.4289,  -0.8812,  -0.2444,  -0.1239
flatness_thr_   :   0.0152,  0.0085,  0.0013,  0.0053
Input Point Cloud Size:         18653
Ground Point Cloud Size:        5352
Non-Ground Point Cloud Size:    13301
Timer took 7.17513ms
@seungjae24
Copy link
Member

seungjae24 commented Jun 7, 2023

Hi @Nothand0212,

It is true that the sensor height is updated in an adaptive manner. You can disable the update of sensor_height variable in the source code, which is written as below.

params_.sensor_height = -update_mean;

Also, it might be great if you share your data collected using boat. It looks very interesting to try Patchwork++ in another platform because I have tested my algorithm only for autonomous vehicles, mobile robots, or quadruped robots. If you don't mind to share your data, please send me an e-mail! The e-mail address is written in the end of README. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants