Fix Doxygen warnings in multiple headers files

cpp/doxygen/Doxyfile

@@ -818,6 +818,12 @@ INPUT                  = main_page.md \
                          regex.md \
                          unicode.md \
                          ../include \
+                         ../include/cudf_test/column_wrapper.hpp \
+                         ../include/cudf_test/column_utilities.hpp \
+                          ../include/cudf_test/iterator_utilities.hpp \
+                          ../include/cudf_test/table_utilities.hpp \
+                          ../include/cudf_test/type_lists.hpp \
+                          ../include/cudf_test/type_list_utilities.hpp \
                          ../libcudf_kafka/include
 
 # This tag can be used to specify the character encoding of the source files
@@ -881,7 +887,7 @@ EXCLUDE_SYMLINKS       = NO
 # Note that the wildcards are matched against the file with absolute path, so to
 # exclude all test directories for example use the pattern */test/*
 
-EXCLUDE_PATTERNS       = */nvtx/* */detail/*
+EXCLUDE_PATTERNS       = */nvtx/* */detail/* */cudf_test/*
 
 # The EXCLUDE_SYMBOLS tag can be used to specify one or more symbol names
 # (namespaces, classes, functions, etc.) that should be excluded from the

cpp/include/cudf/column/column_device_view.cuh

@@ -1155,6 +1155,7 @@ struct value_accessor {
 
   /**
    * @brief constructor
+   *
    * @param[in] _col column device view of cudf column
    */
   value_accessor(column_device_view const& _col) : col{_col}
@@ -1244,6 +1245,7 @@ struct pair_accessor {
 
   /**
    * @brief constructor
+   *
    * @param[in] _col column device view of cudf column
    */
   pair_accessor(column_device_view const& _col) : col{_col}
@@ -1284,6 +1286,7 @@ struct pair_rep_accessor {
 
   /**
    * @brief constructor
+   *
    * @param[in] _col column device view of cudf column
    */
   pair_rep_accessor(column_device_view const& _col) : col{_col}
@@ -1317,6 +1320,7 @@ struct mutable_value_accessor {
 
   /**
    * @brief constructor
+   *
    * @param[in] _col mutable column device view of cudf column
    */
   mutable_value_accessor(mutable_column_device_view& _col) : col{_col}

cpp/include/cudf/copying.hpp

@@ -86,6 +86,7 @@ std::unique_ptr<table> gather(
 
 /**
  * @brief Reverses the rows within a table.
+ *
  * Creates a new table that is the reverse of @p source_table.
  * Example:
  * ```
@@ -103,6 +104,7 @@ std::unique_ptr<table> reverse(
 
 /**
  * @brief Reverses the elements of a column
+ *
  * Creates a new column that is the reverse of @p source_column.
  * Example:
  * ```

cpp/include/cudf/io/parquet.hpp

@@ -528,15 +528,20 @@ class parquet_writer_options {
   /**
    * @brief Returns the maximum uncompressed page size, in bytes. If set larger than the row group
    * size, then this will return the row group size.
+   *
+   * @return Maximum uncompressed page size, in bytes
    */
   auto get_max_page_size_bytes() const
   {
     return std::min(_max_page_size_bytes, get_row_group_size_bytes());
   }
 
   /**
-   * @brief Returns maximum page size, in rows. If set larger than the row group size, then this
-   * will return the row group size.
+   * @brief Returns maximum page size, in rows. 
+   *
+   * If set larger than the row group size, then this will return the row group size.
+   *
+   * @return Maximum page size, in rows
    */
   auto get_max_page_size_rows() const
   {
@@ -638,6 +643,8 @@ class parquet_writer_options {
 
   /**
    * @brief Sets the maximum uncompressed page size, in bytes.
+   *
+   * @param size_bytes Maximum uncompressed page size, in bytes to set
    */
   void set_max_page_size_bytes(size_t size_bytes)
   {
@@ -647,6 +654,8 @@ class parquet_writer_options {
 
   /**
    * @brief Sets the maximum page size, in rows.
+   *
+   * @param size_rows Maximum page size, in rows to set
    */
   void set_max_page_size_rows(size_type size_rows)
   {
@@ -981,6 +990,8 @@ class chunked_parquet_writer_options {
   /**
    * @brief Returns maximum uncompressed page size, in bytes. If set larger than the row group size,
    * then this will return the row group size.
+   *
+   * @return Maximum uncompressed page size, in bytes
    */
   auto get_max_page_size_bytes() const
   {
@@ -990,6 +1001,8 @@ class chunked_parquet_writer_options {
   /**
    * @brief Returns maximum page size, in rows. If set larger than the row group size, then this
    * will return the row group size.
+   *
+   * @return Maximum page size, in rows
    */
   auto get_max_page_size_rows() const
   {
@@ -1065,6 +1078,8 @@ class chunked_parquet_writer_options {
 
   /**
    * @brief Sets the maximum uncompressed page size, in bytes.
+   *
+   * @param size_bytes Maximum uncompressed page size, in bytes to set
    */
   void set_max_page_size_bytes(size_t size_bytes)
   {
@@ -1074,6 +1089,8 @@ class chunked_parquet_writer_options {
 
   /**
    * @brief Sets the maximum page size, in rows.
+   *
+   * @param size_rows The maximum page size, in rows to set
    */
   void set_max_page_size_rows(size_type size_rows)
   {

cpp/include/cudf/io/types.hpp

@@ -519,6 +519,7 @@ class column_in_metadata {
 
   /**
    * @brief Gets the explicitly set nullability for this column.
+   *
    * @throws If nullability is not explicitly defined for this column.
    *         Check using `is_nullability_defined()` first.
    * @return Boolean indicating whether this column is nullable
@@ -549,6 +550,7 @@ class column_in_metadata {
 
   /**
    * @brief Get the decimal precision that was set for this column.
+   *
    * @throws If decimal precision was not set for this column.
    *         Check using `is_decimal_precision_set()` first.
    * @return The decimal precision that was set for this column
@@ -564,6 +566,7 @@ class column_in_metadata {
 
   /**
    * @brief Get the parquet field id that was set for this column.
+   *
    * @throws If parquet field id was not set for this column.
    *         Check using `is_parquet_field_id_set()` first.
    * @return The parquet field id that was set for this column

cpp/include/cudf/ipc.hpp

@@ -1,15 +1,55 @@
+/*
+ * Copyright (c) 2022, NVIDIA CORPORATION.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#pragma once
+
 #include <arrow/api.h>
 #include <arrow/gpu/cuda_api.h>
 #include <arrow/io/memory.h>
 #include <arrow/ipc/api.h>
 
+/**
+ * @brief Reads Message objects from cuda buffer source
+ *
+ */
 class CudaMessageReader : arrow::ipc::MessageReader {
  public:
+  /**
+   * @brief Construct a new Cuda Message Reader object from a cuda buffer stream
+   *
+   * @param stream The cuda buffer reader stream
+   * @param schema The schema of the stream
+   */
   CudaMessageReader(arrow::cuda::CudaBufferReader* stream, arrow::io::BufferReader* schema);
 
+  /**
+   * @brief Open stream from source.
+   *
+   * @param stream The cuda buffer reader stream
+   * @param schema The schema of the stream
+   * @return arrow::ipc::MessageReader object
+   */
   static std::unique_ptr<arrow::ipc::MessageReader> Open(arrow::cuda::CudaBufferReader* stream,
                                                          arrow::io::BufferReader* schema);
 
+  /**
+   * @brief Read next Message from the stream.
+   *
+   * @return arrow::ipc::Message object
+   */
   arrow::Result<std::unique_ptr<arrow::ipc::Message>> ReadNextMessage() override;
 
  private:

cpp/include/cudf/scalar/scalar_device_view.cuh

@@ -36,6 +36,7 @@ class scalar_device_view_base {
 
   /**
    * @brief Returns the value type
+   *
    * @returns The value type
    */
   [[nodiscard]] __host__ __device__ data_type type() const noexcept { return _type; }

cpp/include/cudf_test/base_fixture.hpp

@@ -123,7 +123,7 @@ uint64_t random_generator_incrementing_seed();
 template <typename T = cudf::size_type, typename Engine = std::default_random_engine>
 class UniformRandomGenerator {
  public:
-  using uniform_distribution = uniform_distribution_t<T>;
+  using uniform_distribution = uniform_distribution_t<T>;  ///< The uniform distribution type for T.
 
   UniformRandomGenerator() : rng{std::mt19937_64{detail::random_generator_incrementing_seed()}()} {}
 
@@ -144,6 +144,13 @@ class UniformRandomGenerator {
   {
   }
 
+  /**
+   * @brief Construct a new Uniform Random Generator to generate uniformly random booleans
+   *
+   * @param lower ignored
+   * @param upper ignored
+   * @param seed  seed to initialize generator with
+   */
   template <typename TL = T, std::enable_if_t<cudf::is_boolean<TL>()>* = nullptr>
   UniformRandomGenerator(T lower,
                          T upper,
@@ -179,6 +186,10 @@ class UniformRandomGenerator {
     return T{dist(rng)};
   }
 
+  /**
+   * @brief Returns the next random number.
+   * @return generated random number
+   */
   template <typename TL = T, std::enable_if_t<cudf::is_timestamp<TL>()>* = nullptr>
   T generate()
   {

cpp/include/cudf_test/column_wrapper.hpp

@@ -94,7 +94,14 @@ class column_wrapper {
  */
 template <typename From, typename To>
 struct fixed_width_type_converter {
-  // Are the types same - simply copy elements from [begin, end) to out
+  /**
+   * @brief No conversion necessary: Same type, simply copy element to output.
+   *
+   * @tparam FromT Source type
+   * @tparam ToT Target type
+   * @param element Source value
+   * @return The converted target value, same as source value
+   */
   template <typename FromT                                      = From,
             typename ToT                                        = To,
             std::enable_if_t<std::is_same_v<FromT, ToT>, void>* = nullptr>
@@ -103,7 +110,14 @@ struct fixed_width_type_converter {
     return element;
   }
 
-  // Are the types convertible or can target be constructed from source?
+  /**
+   * @brief Convert types if possible, otherwise construct target from source.
+   *
+   * @tparam FromT Source type
+   * @tparam ToT Target type
+   * @param element Source value
+   * @return The converted target value
+   */
   template <
     typename FromT          = From,
     typename ToT            = To,
@@ -115,7 +129,14 @@ struct fixed_width_type_converter {
     return static_cast<ToT>(element);
   }
 
-  // Convert integral values to timestamps
+  /**
+   * @brief Convert integral values to timestamps
+   *
+   * @tparam FromT Source type
+   * @tparam ToT Target type
+   * @param element Source value
+   * @return The converted target `timestamp` value
+   */
   template <
     typename FromT                                                                  = From,
     typename ToT                                                                    = To,
@@ -484,6 +505,11 @@ class fixed_width_column_wrapper : public detail::column_wrapper {
   }
 };
 
+/**
+ * @brief A wrapper for a column of fixed-width elements.
+ *
+ * @tparam Rep The type of the column
+ */
 template <typename Rep>
 class fixed_point_column_wrapper : public detail::column_wrapper {
  public:
@@ -1445,6 +1471,7 @@ class lists_column_wrapper : public detail::column_wrapper {
    * @brief Construct a list column containing a single empty, optionally null row.
    *
    * @param valid Whether or not the empty row is also null
+   * @return A list column containing a single empty row
    */
   static lists_column_wrapper<T> make_one_empty_row_column(bool valid = true)
   {