Markus Vieth

Results 413 comments of Markus Vieth

@EwingKang Sure, no problem. On Ubuntu 22.04, apt would give you PCL 1.12.1. This problem is fixed in PCL 1.13.0 and newer. So using a newer PCL version should solve...

> > Range image > > You should move the key function (in this case virtual destructor) into a CPP file. https://gcc.gnu.org/wiki/VerboseDiagnostics#missing_vtable Thanks for the suggestion, that seems to work....

> This seems useful. Any chance to get this moved forward? I will take another look

I am happy with the changes now. I will merge this after PCL 1.14.1 is released because adding the `num_threads_` variable breaks the ABI, and we don't want an ABI...

@kaipomauli Hello, great that you want to contribute to PCL! I would suggest that we create a duplicate of the `writeBinary` function https://github.com/PointCloudLibrary/pcl/blob/4ab87d43432b5497b715ace7415f799da7be4a77/io/include/pcl/io/ply_io.h#L604 but with `std::ostream &os` instead of a...

This has a large overlap with PR #4901 . I don't want to review the same changes twice. Could you either modify the PRs so that each change is contained...

Hm, yes, I think `PCL_ROOT` should be `/usr`. Not sure why cmake uses the wrong path there. You could try to set `PCL_DIR` manually as described in [this tutorial](https://pcl.readthedocs.io/projects/tutorials/en/latest/using_pcl_pcl_config.html?highlight=PCL_DIR)

BTW I found this bug report: https://bugs.launchpad.net/ubuntu/+source/pcl/+bug/1896118 @perisage Out of interest, do you also use a Singularity container? Or is there anything special/uncommon about your operating system that may cause...

@themightyoarfish Thanks for sharing your solution. I am closing this issue, since it seems pretty clear that it is not a bug in PCL, but in Apple clang.

We maintainers likely don't have the time right now to work on this, but you are of course welcome to do so and submit a pull request. Just a hint...