Skip to content

Depth odometry organized icp #388

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

Merged
merged 589 commits into from
Jan 8, 2022

Conversation

rsoussan
Copy link
Member

@rsoussan rsoussan commented Jan 6, 2022

Added functions to take advantage of the layout of our point cloud being "organized" (having a width and height since it's from a depth camera) to speed up normal estimation. Also added functions for sampling using normal alignments to further speed up ICP.

This branch is off of the open pr " Added Depth Odometry Sweep Tool and Optional ICP-based refinement for Image Feature Depth Odometry #381, so depending on the merge order that can be closed or merged before this one (I couldn't find an option to make the destination that pr, let me know if there's a better way to do this).
This requires new debs to be installed (libpcl-registration and libpcl-keypoints).

@rsoussan rsoussan marked this pull request as ready for review January 6, 2022 06:53
Copy link
Member

@bcoltin bcoltin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure you add new packages to platform too

@rsoussan rsoussan merged commit b2cf8d4 into nasa:develop Jan 8, 2022
@rsoussan rsoussan deleted the depth_odometry_organized_icp branch January 8, 2022 00:27
luisa-mao pushed a commit to luisa-mao/astrobee-1 that referenced this pull request Jul 11, 2023
Added depth odometry sweep tool, optional refinement of the image feature depth odometry estimate, and organized methods taking advantage of the grid like structure of point clouds to speed up computations.
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