Mark Harris

Contributes to #598. Adds a new libcuspatial C++ developer guide.

We would like to improve cuSpatial's developer documentation to make it easier to contribute to cuSpatial. I think we should follow the lead of cuDF, which has a relatively comprehensive...

[FEA] Refactor Spatial Join APIs to header-only

1

Create header-only APIs for the following and refactor existing APIs / tests on top of the header-only API: * `cuspatial::join_quadtree_and_bounding_boxes()` * `cuspatial::quadtree_point_in_polygon()` * `cuspatial::quadtree_point_to_nearest_polyline()` See https://github.com/rapidsai/cuspatial/blob/branch-22.08/cpp/include/cuspatial/spatial_join.hpp

[FEA] refactor point quadtree API to header-only

1

Create header-only APIs for the following and refactor existing APIs / tests on top of the header-only API: * `cuspatial::quadtree_on_points()` See https://github.com/rapidsai/cuspatial/blob/branch-22.08/cpp/include/cuspatial/point_quadtree.hpp

[FEA] Refactor polyline_bounding_boxes to header-only API

1

Create header-only APIs for the following and refactor existing APIs / tests on top of the header-only API: * `cuspatial::polyline_boundingboxes()` See https://github.com/rapidsai/cuspatial/blob/branch-22.08/cpp/include/cuspatial/polyline_bounding_box.hpp

[FEA] Refactor polygon_bounding_boxes to header-only API

1

Create header-only APIs for the following and refactor existing APIs / tests on top of the header-only API: * cuspatial::polygon_bounding_boxes() See https://github.com/rapidsai/cuspatial/blob/branch-22.08/cpp/include/cuspatial/polygon_bounding_box.hpp

[DISCUSSION] Remove dependency on libcudf?

11

We made a decision in the past to base the libcuspatial API on libcudf. Specifically, `cuspatial::` APIs take `cudf::column` and `cudf::table` inputs. Internally, these APIs use `cudf::type_dispatcher` to dispatch on...

[FEA] Use `cudf::table` and `DataFrame` in cuSpatial APIs

2

**Is your feature request related to a problem? Please describe.** Use `cudf::table` (and DataFrame in Python) type for inputs and outputs where it makes sense. **Describe the solution you'd like**...

[DOC] Docs say to build libcudf doxygen using `make docs_cudf`, which doesn't work.

3

## Report incorrect documentation **Location of incorrect documentation** cudf/cpp/doxygen/developer_guide/DOCUMENTATION.md **Describe the problems or issues found in the documentation** > The libcudf documentation can also be built using `make docs_cudf` from...

[DISCUSSION] Improve C++ source and header organization

1

**Is your feature request related to a problem? Please describe.** I think we should organize our headers and source better. To make implementations, tests, and benchmarks easier to find and...