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

Using a guess transformation in GICP alignment leads to wrong results. #754

Closed
saarnold opened this issue Jun 19, 2014 · 4 comments · Fixed by #887
Closed

Using a guess transformation in GICP alignment leads to wrong results. #754

saarnold opened this issue Jun 19, 2014 · 4 comments · Fixed by #887

Comments

@saarnold
Copy link
Contributor

Using a guess transformation, other than the identity, for the alignment with GICP leads to a wrong results.

It is very easy to test, by transforming the target pointcloud somewhere and using the same transformation as guess. This should lead to the same result, but it doesn't.

You can find a corresponding test case in my clone of PCL.

@VictorLamoine
Copy link
Contributor

Related: issue #818

@VictorLamoine
Copy link
Contributor

Appart from the fact that you have to use the inverse of the transformation as a guess; you are right.
Please test my pull request and comment if it seems right to you.

@saarnold
Copy link
Contributor Author

Sry for the delay, I didn't had the time to check your commit until now.

In my test case I was pre-transforming the target pointcloud, in that case the same transformation has to be used as guess, not its inverse. Since the input (or source) pointcloud will be aligned to the fixed target pointcloud.

Your commit transforms the output pointcloud at the end of the optimization, which is a fair thing to do, but doesn't solve the mentioned problem.
It is still not possible to use a guess transformation, apart from the identity, in the GICP optimization process.

@SergioRAgostinho
Copy link
Member

Closed with #989

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

Successfully merging a pull request may close this issue.

4 participants