search

cartographer-project / point_cloud_viewer

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

Change back to include bounding box for s2 cells. #365

Closed feuerste closed 4 years ago

feuerste commented 4 years ago

Since the meta changes are not in production yet, we can change it back to include the bounding box for both s2 cells and octrees.

nnmm commented 4 years ago

Should the bounding_box method be part of the PointCloud trait?

feuerste commented 4 years ago

Should the bounding_box method be part of the PointCloud trait?

Yes, makes sense, I moved it there.

feuerste commented 4 years ago

Thanks for the review!