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

Differences between KDTREE and DIRECT7 #24

Closed
LandauR opened this issue Mar 26, 2021 · 5 comments
Closed

Differences between KDTREE and DIRECT7 #24

LandauR opened this issue Mar 26, 2021 · 5 comments

Comments

@LandauR
Copy link

LandauR commented Mar 26, 2021

Hi,

Sorry to bother you. Thanks to your great package, I have ran a version of robot localization successfully with NDT in ndt_omp. During my tests, I found something different in performance between KDTREE and DIRECT7, but I found it difficult to explain.

I ran a diff-driven robot with a 2d laser in a static environment on the same path with same 2d map. I ran my robot with DIRECT7, the laser scan did not match the map well while I ran with KDTREE, the laser scan match better.

Here are two pictures of matching with KDTREE and DIRECT7.

KDTREE match below.
IMG_20210326_150125

DIRECT7 match below.
IMG_20210326_150748

I really can not think about a reason to explain this phenomenon. I would be so grateful if you could offer some help on it.

Best regards.

@koide3
Copy link
Owner

koide3 commented Mar 29, 2021

Did you see the same issue with DIRECT1 as well?

@LandauR
Copy link
Author

LandauR commented Mar 29, 2021

Yes, DIRECT1 performed similar with DIRECT7.

@koide3
Copy link
Owner

koide3 commented Mar 30, 2021

Hmm, can you try a small transformation_epsilon_ to see if it's a problem of convergence parameters? In addition to that, please check if the package is the latest version. Older versions have some problems in line search.

@LandauR
Copy link
Author

LandauR commented Mar 2, 2023

We have just found whrere the problem is. It turns out it has noting to do with search method.

@LandauR LandauR closed this as completed Mar 2, 2023
@hiyyg
Copy link

hiyyg commented Apr 17, 2023

We have just found whrere the problem is. It turns out it has noting to do with search method.

May I ask what the actual problem is?

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

4 participants
@koide3 @LandauR @hiyyg and others