Skip to content

Add OpenGR Wrapper for Multiple Point Cloud Registration #4963

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

Open
wants to merge 30 commits into
base: master
Choose a base branch
from

Conversation

beautifulv0id
Copy link

@beautifulv0id beautifulv0id commented Aug 28, 2020

Rationale

CGAL recently added wrappers to their library to enable global and local point-cloud registration using the OpenGR library. In the current version, registration can only be performed between two point-clouds.

This small feature adds a wrapper for the OpenGR library to Point Set Processing that computes global registration of multiple point clouds as described in the following paper: Global Registration of Multiple Point Clouds Using Semidefinite Programming, K. N. Chaudhury, Y. Khoo, and A. Singer, SIAM J. Optim., 25(1), 468501. (34 pages).

Note that the above-mentioned approach assumes that the correspondences between the local coordinates and the global coordinates are known. It is therefore the responsibility of the user to provide these correspondences.

Summary of API Changes

The following two functions were added to the Point_set_processing_3 package:

  • CGAL::OpenGR::compute_registration_transformations,
  • CGAL::OpenGR::register_point_clouds.

Changes

An example that shows the usage of the API: OpenGR GRET-SDP Example.

Links to the reference manual of the added functions:

Status

The code depends on this version of the OpenGR library that implements the above-mentioned registration algorithm. A pull request was opened to the official OpenGR github repository that is waiting to be merged.

Release Management

@beautifulv0id beautifulv0id changed the title Gsoc2020 point set processing 3 global registration wrapper felix Add OpenGR Wrapper for Multiple Point Cloud Registration Aug 28, 2020
@sloriot sloriot added gsoc gsoc-2020 Pkg::Point_set_processing_3 Not yet approved The feature or pull-request has not yet been approved. Small feature labels Aug 31, 2020
@lrineau lrineau requested a review from sgiraudot August 31, 2020 07:30
@MaelRL MaelRL added this to the 5.3-beta milestone Oct 14, 2020
@MaelRL MaelRL requested review from sgiraudot and removed request for sgiraudot February 8, 2021 17:34
@maxGimeno
Copy link
Contributor

There are conflicts with master

@sgiraudot
Copy link
Contributor

The OpenGR PR on which this PR depends is not integrated yet, I suggest we consider this feature as stalled until it's done.

@nmellado do you have an estimation of when Felix's PR will be integrated on your side? Can you ping us when it is the case so we can reactivate this PR?

@maxGimeno maxGimeno added the rm: not for next release Indicate to the release team that a PR should not be merged before the next release branch is forked label Mar 3, 2021
@maxGimeno maxGimeno modified the milestones: 5.3-beta, 5.4-beta Mar 3, 2021
@lrineau lrineau removed the rm: not for next release Indicate to the release team that a PR should not be merged before the next release branch is forked label Jul 7, 2021
@sloriot
Copy link
Member

sloriot commented Sep 29, 2021

ping @nmellado

@sloriot sloriot modified the milestones: 5.4-beta, 5.5-beta1 Sep 30, 2021
@nmellado
Copy link

nmellado commented Feb 8, 2022

Hi,
Just to let you know that I restarted to work on this project, on the OpenGR side.
Hopefully we may have a proposal in the upcoming weeks.

@MaelRL MaelRL modified the milestones: 5.5-beta, 5.6-beta Mar 28, 2022
@MaelRL MaelRL modified the milestones: 5.6-beta, 5.7-beta Mar 31, 2023
@janetournois janetournois modified the milestones: 6.0-beta, 6.1-beta May 16, 2024
@MaelRL MaelRL modified the milestones: 6.1-beta, 6.2-beta Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants