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

When will the wrappers of PCL be released? #41

Closed
RaymondZuo301 opened this issue Aug 2, 2017 · 8 comments
Closed

When will the wrappers of PCL be released? #41

RaymondZuo301 opened this issue Aug 2, 2017 · 8 comments

Comments

@RaymondZuo301
Copy link

When will the wrappers of PCL be released?

@nmellado
Copy link
Owner

nmellado commented Aug 2, 2017

Hi,

This wrapper is part of my mid-term roadmap.
I don't have a clear deadline for this, but hopefully it will be done by the end of the year.

Of course if many people request it I can try to speed this up.

@xinkang
Copy link
Contributor

xinkang commented Aug 2, 2017

I love this idea. It will be a great feature and probably a good way to get the algorithm exposed to a larger population.

@RaymondZuo301
Copy link
Author

Thanks for your work

@nmellado nmellado added this to the Ecosystem and Wrappers milestone Aug 11, 2017
@nmellado nmellado mentioned this issue Aug 24, 2017
@nmellado
Copy link
Owner

nmellado commented Aug 24, 2017

Here you go ;), WIP in #45

@nmellado nmellado added the WIP label Sep 6, 2017
@nmellado
Copy link
Owner

@zfrxiaxia @xinkang ,
A first version of the PCLWrapper is available in branch v1.1.3RC, if you want to test it....

@RaymondZuo301
Copy link
Author

#45 I got this from Super4PCS-PCLWrapper
////////////////////////////////////////////////////////////////////////////////////
./Super4PCS-PCLWrapper hippo1.obj hippo2.obj

Loading point clouds...
Failed to find match for field 'curvature'.
Failed to find match for field 'curvature'.
Starting alignment...
norm_max_dist: 5
Initial LCP: 1
Segmentation fault (core dumped)
////////////////////////////////////////////////////////////////////////////////////

@nmellado
Copy link
Owner

nmellado commented Sep 15, 2017

@zfrxiaxia , thanks for testing.

Sorry, I forgot to give you proper run instructions.
Segfault is now fixed 0f43688.

To test the wrapper, please run:
../bin/Super4PCS-PCLWrapper ../assets/hippo1.obj ../assets/hippo2.obj -o 0.7 -d 0.01 -t 1000 -n 200
(paths to binary and assets must be adapted)

@nmellado
Copy link
Owner

Done with new release v1.1.3.
Closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants