search

cartographer-project / point_cloud_viewer

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

a binary for merging subquadtrees into a complete quadtree. #440

Closed krzesi-mir closed 4 years ago

krzesi-mir commented 4 years ago

LGTM The only remaining thing is that we have lots of expect(...)s and unwrap()s... we may consider removing them... But this could be in a separate PR, too.

Yeah, I agree with, you. Will do it in another PR.

krzesi-mir commented 4 years ago

Thanks for the review!