cartographer-project / point_cloud_viewer

View billions of points in your browser.
Apache License 2.0
339 stars 98 forks source link

Add a test for PointCulling/ConvexPolyhedron consistency #452

Closed nnmm closed 4 years ago

nnmm commented 4 years ago

We have two implementations of a containment predicate on points: One via the PointCulling trait, and one via the ConvexPolyhedron trait. They should behave the same.

This also fixes a bug in the tests where the Aabb was equal to the entire point cloud's bbox.

nnmm commented 4 years ago

Thanks for reviewing!