Consolidate bounding box code

[harrism]

Fixes #746. Replaces the header-only trajectory_bounding_boxes with point_bounding_boxes and reuses it to implement linestring_bounding_boxes and polygon_bounding_boxes.

By providing a single header-only function point_bounding_boxes that forms the implementation of trajectory_bounding_boxes, linestring_bounding_boxes, and polygon_bounding_boxes, this PR also closes closes 567 and closes 568.

It does not, as point_bounding_boxes takes IDs, not polygon and ring offsets as the other APIs do. So we can't quite mark those as fixed.

Checklist

• I am familiar with the Contributing Guidelines.
• New or existing tests cover these changes.
• The documentation is up to date with these changes.

[robertmaynard]

I believe that cpp/tests/CMakeLists.txt needs to be updated since we renamed experimental/trajectory/trajectory_bounding_boxes_test.cu

[harrism]

I believe that cpp/tests/CMakeLists.txt needs to be updated since we renamed experimental/trajectory/trajectory_bounding_boxes_test.cu

You must have missed where I updated it? It's now called POINT_BOUNDING_BOXES_TEST_EXP

[isVoid]

Good cleanup!

[trxcllnt]

rerun tests

[harrism]

Ugh, I wish pre-commit would run for github-ui commits.

[robertmaynard]

Aprroving CMake changes

[harrism]

@gpucibot merge