Lars Glud

Results 131 comments of Lars Glud

Yeah, I could have looked there but I just added locally until it would build. Seems some could be removed on vcpkg. Ie. datetime.

I have had trouble when having multiple versions of cuda installed before as well. Generally we use the enable_language(cuda) option - so you could also try to make a minimal...

Hey @alexnavtt I just notized this comment at the documentation: Note The code is stateful as we do not expect this class to be multicore parallelized. Please look at [NormalEstimationOMP](https://pointclouds.org/documentation/classpcl_1_1_normal_estimation_o_m_p.html)...

As per having both PrincipalCurvaturesEstimation and a PrincipalCurvaturesEstimationOMP - I think the latest direction was moving the algorithm to be only in the base class and then in the OMP...

The above is only suggestions as we try to move towards https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-default-member-init.html

Look at the same already answered issue here #723. You need to host it over https so there is a security context.

Friendly ping @mvieth 😄