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

Added a transformation estimation method that uses point-to-plane LM and per-correspondence weights. #88

Merged
merged 2 commits into from
May 24, 2013

Conversation

aichim
Copy link
Member

@aichim aichim commented May 17, 2013

Jochen,

Could you please have a quick look, as it is not nice to accept my own pull requests :-) ?

Thanks!

@ghost ghost assigned jspricke May 17, 2013
@jspricke
Copy link
Member

Hi Alex,

do you still want to send a new version of this, or should I just merge it?

Cheers Jochen

@aichim
Copy link
Member Author

aichim commented May 23, 2013

Hi Jochen,

I did some fixes as you suggested, but not a bigger re-haul of all the transformation estimation classes. We should do that after the release of 1.7 I think, and after I add some other methods.

So you could merge it for now, if you are happy with the current state.

Cheers,
Alex

jspricke added a commit that referenced this pull request May 24, 2013
Added a transformation estimation method that uses point-to-plane LM and per-correspondence weights.
@jspricke jspricke merged commit c871641 into PointCloudLibrary:master May 24, 2013
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 this pull request may close these issues.

2 participants