Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions cpp/include/cudf/column/column_device_view_base.cuh
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ class alignas(16) column_device_view_base {
* This function will only participate in overload resolution if `is_rep_layout_compatible<T>()`
* or `std::is_same_v<T,void>` are true.
*
* @tparam The type to cast to
* @tparam T The type to cast to
* @return Typed pointer to underlying data
*/
template <typename T = void,
Expand Down Expand Up @@ -638,7 +638,7 @@ class alignas(16) mutable_column_device_view_core : public detail::column_device
* a column, and instead, accessing the elements should be done via
* `data<T>()`.
*
* @tparam The type to cast to
* @tparam T The type to cast to
* @return Typed pointer to underlying data
*/
template <typename T = void,
Expand Down
4 changes: 2 additions & 2 deletions cpp/include/cudf/column/column_view.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ class column_view_base {
* This function will only participate in overload resolution if `is_rep_layout_compatible<T>()`
* or `std::is_same_v<T,void>` are true.
*
* @tparam The type to cast to
* @tparam T The type to cast to
Comment thread
igorpeshansky marked this conversation as resolved.
* @return Typed pointer to underlying data
*/
template <typename T = void,
Expand Down Expand Up @@ -560,7 +560,7 @@ class mutable_column_view : public detail::column_view_base {
* @note It should be rare to need to access the `head<T>()` allocation of a
* column, and instead, accessing the elements should be done via `data<T>()`.
*
* @tparam The type to cast to
* @tparam T The type to cast to
* @return Typed pointer to underlying data
*/
template <typename T = void,
Expand Down
4 changes: 2 additions & 2 deletions cpp/include/cudf/detail/iterator.cuh
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* SPDX-FileCopyrightText: Copyright (c) 2019-2026, NVIDIA CORPORATION.
* SPDX-FileCopyrightText: Copyright (c) 2019-2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
* SPDX-License-Identifier: Apache-2.0
*/

Expand Down Expand Up @@ -324,7 +324,7 @@ CUDF_HOST_DEVICE auto inline make_validity_iterator(column_device_view const& co
*
* For `p = *(iter + i)`, `p` is the validity of the scalar.
*
* @tparam bool unused. This template parameter exists to enforce the same
* @tparam safe unused. This template parameter exists to enforce the same
* template interface as @ref make_validity_iterator(column_device_view const&).
* @param scalar_value The scalar to iterate
* @return auto Iterator that returns scalar validity
Expand Down
10 changes: 1 addition & 9 deletions cpp/include/cudf/detail/utilities/vector_factories.hpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* SPDX-FileCopyrightText: Copyright (c) 2021-2026, NVIDIA CORPORATION.
* SPDX-FileCopyrightText: Copyright (c) 2021-2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
* SPDX-License-Identifier: Apache-2.0
*/

Expand Down Expand Up @@ -102,7 +102,6 @@ rmm::device_uvector<std::remove_cv_t<T>> make_device_uvector_async(
* @note This function does not synchronize `stream`.
*
* @tparam Container The type of the container to copy from
* @tparam T The type of the data to copy
* @param c The input host container from which to copy
* @param stream The stream on which to allocate memory and perform the copy
* @param mr The memory resource to use for allocating the returned device_uvector
Expand Down Expand Up @@ -166,7 +165,6 @@ rmm::device_uvector<std::remove_cv_t<T>> make_device_uvector_async(
* @note This function does not synchronize `stream`.
*
* @tparam Container The type of the container to copy from
* @tparam T The type of the data to copy
* @param c The input device container from which to copy
* @param stream The stream on which to allocate memory and perform the copy
* @param mr The memory resource to use for allocating the returned device_uvector
Expand Down Expand Up @@ -210,7 +208,6 @@ rmm::device_uvector<T> make_device_uvector(host_span<T const> source_data,
* @note This function synchronizes `stream`.
*
* @tparam Container The type of the container to copy from
* @tparam T The type of the data to copy
* @param c The input host container from which to copy
* @param stream The stream on which to allocate memory and perform the copy
* @param mr The memory resource to use for allocating the returned device_uvector
Expand Down Expand Up @@ -273,7 +270,6 @@ rmm::device_uvector<T> make_device_uvector(device_span<T const> source_data,
* @note This function synchronizes `stream`.
*
* @tparam Container The type of the container to copy from
* @tparam T The type of the data to copy
* @param c The input device container from which to copy
* @param stream The stream on which to allocate memory and perform the copy
* @param mr The memory resource to use for allocating the returned device_uvector
Expand Down Expand Up @@ -316,7 +312,6 @@ std::vector<std::remove_cv_t<T>> make_std_vector_async(device_span<T> v,
* @note This function synchronizes `stream` after the copy.
*
* @tparam Container The type of the container to copy from
* @tparam T The type of the data to copy
* @param c The input device container from which to copy
* @param stream The stream on which to perform the copy
* @return The data copied to the host
Expand Down Expand Up @@ -355,7 +350,6 @@ std::vector<T> make_std_vector(device_span<T const> v, rmm::cuda_stream_view str
* @note This function synchronizes `stream`.
*
* @tparam Container The type of the container to copy from
* @tparam T The type of the data to copy
* @param c The input device container from which to copy
* @param stream The stream on which to perform the copy
* @return The data copied to the host
Expand Down Expand Up @@ -430,7 +424,6 @@ host_vector<T> make_host_vector_async(device_span<T const> v, rmm::cuda_stream_v
* using a pinned memory resource.
*
* @tparam Container The type of the container to copy from
* @tparam T The type of the data to copy
* @param c The input device container from which to copy
* @param stream The stream on which to perform the copy
* @return The data copied to the host
Expand Down Expand Up @@ -470,7 +463,6 @@ host_vector<T> make_host_vector(device_span<T const> v, rmm::cuda_stream_view st
* @note This function synchronizes `stream` after the copy.
*
* @tparam Container The type of the container to copy from
* @tparam T The type of the data to copy
* @param c The input device container from which to copy
* @param stream The stream on which to perform the copy
* @return The data copied to the host
Expand Down
4 changes: 0 additions & 4 deletions cpp/include/cudf/interop.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -323,8 +323,6 @@ class arrow_column {
/**
* @brief Get a view of the column data
*
* @param stream CUDA stream used for device memory operations and kernel launches
* @param mr Device memory resource used for any allocations during conversion
* @return A view of the column data
*/
[[nodiscard]] column_view view() const;
Comment thread
igorpeshansky marked this conversation as resolved.
Expand Down Expand Up @@ -443,8 +441,6 @@ class arrow_table {
/**
* @brief Get a view of the table data
*
* @param stream CUDA stream used for device memory operations and kernel launches
* @param mr Device memory resource used for any allocations during conversion
* @return A view of the table data
*/
[[nodiscard]] table_view view() const;
Expand Down
16 changes: 8 additions & 8 deletions cpp/include/cudf/utilities/traits.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -377,8 +377,8 @@ bool is_floating_point(data_type type);
* @brief Indicates whether `T` is a std::byte type.
*
* @tparam T The type to verify
* @return true `type` is std::byte
* @return false `type` is not std::byte
* @return true `T` is std::byte
* @return false `T` is not std::byte
*/
template <typename T>
constexpr inline bool is_byte()
Expand All @@ -389,9 +389,9 @@ constexpr inline bool is_byte()
/**
* @brief Indicates whether `T` is a Boolean type.
*
* @param type The `data_type` to verify
* @return true `type` is Boolean
* @return false `type` is not Boolean
* @tparam T The type to verify
* @return true `T` is Boolean
* @return false `T` is not Boolean
*/
template <typename T>
constexpr inline bool is_boolean()
Expand Down Expand Up @@ -642,9 +642,9 @@ bool is_compound(data_type type);
* can have an arbitrarily deep list of descendants of the same
* type. Strings are not a nested type, but lists are.
*
* @param T The type to verify
* @return true T is a nested type
* @return false T is not a nested type
* @tparam T The type to verify
* @return true `T` is a nested type
* @return false `T` is not a nested type
*/
template <typename T>
CUDF_HOST_DEVICE constexpr inline bool is_nested()
Expand Down
4 changes: 2 additions & 2 deletions cpp/include/cudf/utilities/type_dispatcher.hpp
Comment thread
igorpeshansky marked this conversation as resolved.
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ struct id_to_type_impl {
* ```
* static_assert(std::is_same<int32_t, id_to_type<id_type::INT32>);
* ```
* @tparam t The `cudf::type_id` to map
* @tparam Id The `cudf::type_id` to map
Comment thread
igorpeshansky marked this conversation as resolved.
*/
template <cudf::type_id Id>
using id_to_type = typename id_to_type_impl<Id>::type;
Expand Down Expand Up @@ -443,7 +443,7 @@ using scalar_device_type_t = typename type_to_scalar_type_impl<T>::ScalarDeviceT
* lambda must be the same, else there will be a compiler error as you would be
* trying to return different types from the same function.
*
* @tparam id_to_type_impl Maps a `cudf::type_id` its dispatched C++ type
* @tparam IdTypeMap Maps a `cudf::type_id` to its dispatched C++ type
* @tparam Functor The callable object's type
* @tparam Ts Variadic parameter pack type
* @param dtype The `cudf::data_type` whose `id()` determines which template
Expand Down
5 changes: 3 additions & 2 deletions cpp/include/cudf_test/column_wrapper.hpp
Comment thread
igorpeshansky marked this conversation as resolved.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* SPDX-FileCopyrightText: Copyright (c) 2019-2026, NVIDIA CORPORATION.
* SPDX-FileCopyrightText: Copyright (c) 2019-2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
* SPDX-License-Identifier: Apache-2.0
*/

Expand Down Expand Up @@ -920,7 +920,8 @@ class strings_column_wrapper : public detail::column_wrapper {
* This class handles fixed-width type keys.
*
* @tparam KeyElementTo Specify a fixed-width type for the key values of the dictionary
* @tparam SourceElementTo For converting fixed-width values to the KeyElementTo
* @tparam SourceElementT The fixed-width element type that is used to create elements of type
* `KeyElementTo`
*/
template <typename KeyElementTo, typename SourceElementT = KeyElementTo>
class dictionary_column_wrapper : public detail::column_wrapper {
Expand Down
7 changes: 3 additions & 4 deletions cpp/include/cudf_test/timestamp_utilities.cuh
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* SPDX-FileCopyrightText: Copyright (c) 2019-2024, NVIDIA CORPORATION.
* SPDX-FileCopyrightText: Copyright (c) 2019-2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
* SPDX-License-Identifier: Apache-2.0
*/

Expand All @@ -26,9 +26,8 @@ using time_point_ms =
* The period is inferred from `count` and difference between `start`
* and `stop`.
*
* @tparam Rep The arithmetic type representing the number of ticks
* @tparam Period A cuda::std::ratio representing the tick period (i.e. the
*number of seconds per tick)
* @tparam T The timestamp type of the generated column's elements
* @tparam nullable Whether the generated column carries a validity mask
* @param count The number of timestamps to create
* @param start The first timestamp as a cuda::std::chrono::time_point
* @param stop The last timestamp as a cuda::std::chrono::time_point
Expand Down
2 changes: 1 addition & 1 deletion cpp/src/rolling/detail/range_utils.cuh
Original file line number Diff line number Diff line change
Expand Up @@ -604,7 +604,7 @@ struct range_window_clamper {
/**
* @brief Is the given type supported as an orderby column.
*
* @tparam The type of the elements of the orderby column.
* @tparam OrderbyT The type of the elements of the orderby column.
*/
template <typename OrderbyT>
static constexpr bool is_supported()
Expand Down
2 changes: 1 addition & 1 deletion cpp/src/strings/count_matches.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ namespace detail {
* also need the device program for other work (e.g. extraction) should build it once
* and pass it here to avoid a redundant device program build.
*
* @tparam The regex prog device instance used for this API
* @tparam ProgDevice The regex prog device instance used for this API
* @param d_strings Device view of the input strings column
* @param d_prog Device regex program to evaluate on each string
* @param strings_count Number of strings (and rows in the output column)
Expand Down
Loading