Address @vyasr PR comments.

cpp/include/cuspatial/distances/linestring_distance.hpp

@@ -88,9 +88,9 @@ namespace cuspatial {
  * @param linestring1_offsets Indices of the first point of the first linestring of each pair
  * @param linestring1_points_x x-components of points in the first linestring of each pair
  * @param linestring1_points_y y-component of points in the first linestring of each pair
- * @param linestring2_offsets Indices of the first point of the second linestring of each pair
- * @param linestring2_points_x x-component of points in the first linestring of each pair
- * @param linestring2_points_y y-component of points in the first linestring of each pair
+ * @param linestring2_offsets Indices of the second point of the second linestring of each pair
+ * @param linestring2_points_x x-component of points in the second linestring of each pair
+ * @param linestring2_points_y y-component of points in the second linestring of each pair
  * @param mr Device memory resource used to allocate the returned column's device memory
  * @return A column of shortest distances between each pair of linestrings
  *

cpp/include/cuspatial/experimental/type_utils.hpp

@@ -91,9 +91,9 @@ auto make_vec_2d_iterator(FirstIter first, SecondIter second)
  *
  * Interleaves longitude and latitude from separate iterators into a single iterator to lon/lat
  * coordinates.
- * @tparam FirstIter Iterator type to the first component of `lonlat_2d`. Must meet the requirements
+ * @tparam FirstIter Iterator type to the first component (the longitude) of `lonlat_2d`. Must meet the requirements
  * of [LegacyRandomAccessIterator][LinkLRAI] and be device-accessible.
- * @tparam SecondIter Iterator type to the second component of `lonlat_2d`. Must meet the
+ * @tparam SecondIter Iterator type to the second component (the latitude) of `lonlat_2d`. Must meet the
  * requirements of [LegacyRandomAccessIterator][LinkLRAI] and be device-accessible.
  * @param first Iterator to beginning of `lonlat_2d::x`
  * @param second Iterator to beginning of `lonlat_2d::y`

cpp/include/cuspatial/spatial_join.hpp

@@ -52,8 +52,8 @@ namespace cuspatial {
  * @throw cuspatial::logic_error If max_depth is less than 1 or greater than 15
  *
  * @return A cudf table with two columns:
- * poly_offset - INT32 column of indices for each poly bbox that intersects with the quadtree.
- * quad_offset - INT32 column of indices for each leaf quadrant intersecting with a poly bbox.
+ *   - poly_offset - INT32 column of indices for each poly bbox that intersects with the quadtree.
+ *   - quad_offset - INT32 column of indices for each leaf quadrant intersecting with a poly bbox.
  */
 std::unique_ptr<cudf::table> join_quadtree_and_bounding_boxes(
   cudf::table_view const& quadtree,

cpp/include/cuspatial/types.hpp

@@ -27,7 +27,7 @@
 namespace cuspatial {
 
 /**
- * @addtogroup type_definition_operator
+ * @addtogroup types
  * @{
  */
 

cpp/include/cuspatial/utility/vec_2d.hpp

@@ -20,7 +20,7 @@
 namespace cuspatial {
 
 /**
- * @addtogroup type_definition_operator
+ * @addtogroup types
  * @{
  */
 

cpp/include/doxygen_groups.h

@@ -84,12 +84,10 @@
  * @}
  * @defgroup cuspatial_types Types
  * @{
- *      @defgroup type_definition_operator Definition and Operators
- *      @{
- *          @brief Type declarations for cuspatial
- *          @file types.hpp
- *          @file vec_2d.hpp
- *      @}
+ *      @brief Type declarations for cuspatial
+ *      @file types.hpp
+ *      @file vec_2d.hpp
+ *
  *      @defgroup type_factories Factory Methods
  *      @{
  *          @brief Factory method to create coordinate iterators