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

difference between conventional ndt and this package #1

Closed
weisongwen opened this issue Mar 19, 2018 · 19 comments
Closed

difference between conventional ndt and this package #1

weisongwen opened this issue Mar 19, 2018 · 19 comments

Comments

@weisongwen
Copy link

hi @koide3

thanks for your kind effort on this package, could you tell me what is the difference between this package and conventional ndt ?

Best
Welson

@koide3
Copy link
Owner

koide3 commented Mar 21, 2018

Hi,
The major difference is that this package provides a capability of UKF-based sensor pose estimation. It allows us to start NDT with a good initial guess and makes the scan matching fast and robust.

@weisongwen
Copy link
Author

weisongwen commented Mar 21, 2018

Hi, @koide3 Thanks for your kind reply
have you try to integrate the GPS into the localization?

Best
Welson

@koide3
Copy link
Owner

koide3 commented Mar 22, 2018

Hi,
No, at this time, we've not integrated GPS info with localization. We use GPS at only the mapping step.

@weisongwen
Copy link
Author

Hi, @koide3
thanks for your kind reply, in the mapping step. can this package can do real time mapping?
Thanks
Best
Welson

@koide3
Copy link
Owner

koide3 commented Mar 22, 2018

Hi,
Yes. We confirmed that this package can run real-time on a standard desktop PC. We expect that it can run on a laptop PC as well, but it depends on the setting.

@weisongwen
Copy link
Author

Hi @koide3
Thanks for your kind reply, I mean: if the mapping package "hdl_graph_slam" can run real time?
Thanks
Best
Welson

@koide3
Copy link
Owner

koide3 commented Mar 22, 2018

Hi,
Yes, I meant "hdl_graph_slam" can run real-time.

@weisongwen
Copy link
Author

hi @koide3
thanks for your kind reply,
best
welson

@jeannotes
Copy link

@weisongwen
Hi, did you run the bag? I didn't not get the final result as in the viseos.

@weisongwen
Copy link
Author

Hi, @JSRGFJZ
I have not test the localization package yet,
Best
Welson

@koide3
Copy link
Owner

koide3 commented Apr 2, 2018

Hi, @JSRGFJZ
How did it fail? Did the programs start correctly?

@jeannotes
Copy link

@koide3
Hi, I notice that when I didn't using IMU to predict, the error is very large.WHY?
is it because of only the lidar is one line instead of multi-line? if I have velodyne-16, could it be better?

@weisongwen
Copy link
Author

Hi @JSRGFJZ
I think that you may not give a good initialization when you begin the mapping based localization process.

@koide3
Copy link
Owner

koide3 commented Apr 3, 2018

Hi @JSRGFJZ ,
You are trying to run the bag (hdl_400.bag, 32 line LIDAR), right? I guess the computation power of your PC is insufficient for the default localization parameters. Can you change ndt_neighbor_search_method in hdl_localization.launch to "DIRECT1" and re-try to run the bag? It makes the localization faster significantly.

@jeannotes
Copy link

@koide3
Actually I already set it to DIRECT1, but the result is not good.

@koide3
Copy link
Owner

koide3 commented Apr 10, 2018

@JSRGFJZ
Sorry for the late response. I still can't understand how it fails. Can you try it with a slower playback speed? Like:

rosbag play --clock -r 0.1 hdl_400.bag

@jeannotes
Copy link

@koide3
I already tried that, it still doesn't work.
I didn't use IMU, this should be the reason. but I read your thesis, you said it also could, so should I alter some parameters?

@koide3
Copy link
Owner

koide3 commented Apr 15, 2018

@JSRGFJZ
Yes, it should be able to use it without IMU. But, the lack of IMU makes the localization weak against quick motions. Make me sure that it works with IMU, right?

@jeannotes
Copy link

jeannotes commented Apr 16, 2018 via email

@koide3 koide3 closed this as completed Apr 24, 2018
koide3 pushed a commit that referenced this issue Sep 7, 2020
merge from original source
narutojxl pushed a commit to narutojxl/hdl_localization_noted that referenced this issue Mar 24, 2022
merge from original source
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

3 participants