Norm Evangelista

Results 25 comments of Norm Evangelista

@mvieth Would you consider it worthwhile to convert the many `memset` usages in opennurbs?

> > @mvieth Would you consider it worthwhile to convert the many `memset` usages in opennurbs? > > It would be great to get rid of all the compiler warnings...

Easiest way: - Add `set(CMAKE_EXPORT_COMPILE_COMMANDS ON)` to `CMakeLists.txt` - Run `cmake .` - This generates `compile_commands.json` - Use `run-clang-tidy`, which will look for, and iterate through, `compile_commands.json` and apply the...

> Is there a way to generate `compile_commands.json` without requiring all dependencies to be installed? `set(CMAKE_EXPORT_COMPILE_COMMANDS ON)` just generates the compile database, I'm not sure that it's analyzing or requiring...

> > Is there a way to generate `compile_commands.json` without requiring all dependencies to be installed? > > Yes, as it requires a successful run of CMake. Furthermore clang-tidy requires...

Opened PR #4560, working from the [base modules of the PCL dependency graph](https://user-images.githubusercontent.com/1709142/87282880-fd5cc080-c4f4-11ea-878e-9c7de9c71eb8.png) to keep the LOC changed to a manageable quantity. Future PRs will continue up the dependency graph....

> PS: Builds are failing (Ignore MSVC for a while) Yes, unit tests are failing in `CovarianceSampling.Filters`. So far, I'm only able to resolve it to something in Eigen.

> Hey, sorry but I've not seen the past few updates due to GSoC rush. I'm marking this as "needs more work", but not marking this as draft so you...

@kunaltyagi Do you know how to debug sections under `#pragma omp parallel`? I'm down to one failing unit test (test_poisson). It's failing because in surface/include/pcl/surface/3rdparty/poisson4/multi_grid_octree_data.hpp, between lines 2843 and 2841,...