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

Susan #459

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

Susan #459

wants to merge 2 commits into from

Conversation

nizar-sallem
Copy link
Contributor

Add new SUSAN detectors SusanKeypoint3D operates only on geometric criterion (normals orientation) and SusanKeypoint2D, for comparison purposes, operates on intensities.

Update: Make cases for when the cloud is organized and when not
Update: various fixes and enhancements
@jspricke
Copy link
Member

Same comment as #409 ;).

@jspricke
Copy link
Member

HI @nizar-sallem There is a merge conflict right now, can you have a look? Also, do you have an example application, like in 409? Otherwise I'm fine with merging.

@nizar-sallem
Copy link
Contributor Author

Hi Jochen,

Sorry for the delay was traveling past few days, yes I will have a look and add an application as well.

@SergioRAgostinho
Copy link
Member

@nizar-sallem this, next hackfest.

@SergioRAgostinho SergioRAgostinho added the needs: more work Specify why not closed/merged yet label Aug 22, 2016
@SergioRAgostinho SergioRAgostinho added status: stale needs: author reply Specify why not closed/merged yet and removed needs: more work Specify why not closed/merged yet labels Nov 22, 2017
@stale stale bot closed this Dec 14, 2017
@jspricke jspricke reopened this Dec 18, 2017
@stale stale bot removed the status: stale label Dec 18, 2017
@stale
Copy link

stale bot commented Feb 21, 2020

This pull request has been automatically marked as stale because it hasn't had
any activity in the past 60 days. Commenting or adding a new commit to the
pull request will revert this.

Come back whenever you have time. We look forward to your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs: author reply Specify why not closed/merged yet status: stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants