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

WIP: Point to Plane ICP #3899

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

PranjalSahu
Copy link
Contributor

@PranjalSahu PranjalSahu commented Feb 1, 2023

This is a work-in-progress PR for Point to Plane ICP.
It requires the per-point normal and the Jacobian computation is different compared to Point to Point.
Idea here is to store the normal as data for each point and then use it while calculating Jacobian.

Some references for the implementation:
https://github.com/niosus/notebooks/blob/master/icp.ipynb

PR Checklist

  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)
  • Updated API documentation (or API not changed)
  • Added license to new files (if any)
  • Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
  • Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

@github-actions github-actions bot added type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct area:Registration Issues affecting the Registration module labels Feb 1, 2023
@dzenanz dzenanz marked this pull request as draft February 2, 2023 15:00
@thewtex
Copy link
Member

thewtex commented Feb 6, 2023

CC @ntustison

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:Registration Issues affecting the Registration module type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants