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

(Minor) modifications for the registration example in the RAM article: #1262

Merged
merged 1 commit into from
Jun 24, 2015

Conversation

dirkholz
Copy link
Contributor

  • GICP now allows setting (externally computed) covariance matrices.
    If not set, GICP will compute these as usual.
  • OrganizedFastMesh now accepts a complete quadratic error in the form
    of sigma_d = a_d_d + b*d + c.
  • OrganizedFastMesh now allows to either use the measured depth (z) or
    the actual distance to the point (e.g., for 3D laser scans)
  • from_meshes.h contains two additional inline functions for computing
    approximate local surface normals on polygons and a PolygonMesh,
    respectively, and approximate covariance matrices given points and
    normals (GICP-style).

* GICP now allows setting (externally computed) covariance matrices.
  If not set, GICP will compute these as usual.
* OrganizedFastMesh now accepts a complete quadratic error in the form
  of sigma_d = a*d*d + b*d + c.
* OrganizedFastMesh now allows to either use the measured depth (z) or
  the actual distance to the point (e.g., for 3D laser scans)
* from_meshes.h contains two additional inline functions for computing
  approximate local surface normals on polygons and a PolygonMesh,
  respectively, and approximate covariance matrices given points and
  normals (GICP-style).
@dirkholz
Copy link
Contributor Author

Alex (Ichim) should best know what to do with that ... :)

@andersgb1
Copy link
Contributor

One suggestion: how about an overload for computeApproximateNormals(),
which instead of a point cloud with polygon indices takes a
pcl::PolygonMesh?

On Tue, Jun 16, 2015 at 3:20 PM, Dirk Holz notifications@github.com wrote:

Alex (Ichim) should best know what to do with that ... :)


Reply to this email directly or view it on GitHub
#1262 (comment)
.

@dirkholz
Copy link
Contributor Author

Good point! I'll add some additional functions and proper interfaces once it is in...

@andersgb1
Copy link
Contributor

Awesome

On Thu, Jun 18, 2015 at 11:27 AM, Dirk Holz notifications@github.com
wrote:

Good point! I'll add some additional functions and proper interfaces once
it is in...


Reply to this email directly or view it on GitHub
#1262 (comment)
.

aichim added a commit that referenced this pull request Jun 24, 2015
(Minor) modifications for the registration example in the RAM article:
@aichim aichim merged commit e5a7e46 into PointCloudLibrary:master Jun 24, 2015
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.

3 participants