Kunal Tyagi

Results 187 comments of Kunal Tyagi

1. PCL distributed with OS doesn't come with CUDA enabled. As such there might be conflicting compiler flag compatibility issues. 2. You mentioned PCL 1.10 is used, but the logs...

Please install PCL from source to get access to CUDA. Then use the installed source to compile example. The distributed PCL might be missing library/header files/dependencies for CUDA which is...

Candidate for squash-merge if accepted

Ah, yeah, I couldn't think of a shorter name. Returning the same point as result of search is basically a trivial answer. Ofc the point from the cloud is going...

@peci1 While this effort in PCL is fine, it would also make sense to change the layout in Ouster point format. The issue being: ```cpp float x, y, z; float...

Glad you like this :) Regarding usability: This is a WIP. There are some upstream clang issues (clang doesn't parse ` = delete;`, etc.), some integration issues (no end-to-end cmake)...

This works, but there's tons of work to be done on this to make it usable. However, thoughts? --- I used this as: ``` python doxygen_fixer.py test.cpp clang-format test.cpp ```

The list of doxygen commands is a bit long. And PCL uses both `@command` and `\command` styles. The CommentPragmas line would be a monster. I could create a separate PR...

Both classes could are functionally equivalent to `std::optional`. We can either derive the classes privately, or create the required interfaces. A nice interface would be: * some elements of `optional`...