Skip to content

Conversation

@davidwendt
Copy link
Contributor

Description

Adds a cuda::std::span operator to the cudf::column_view class to allow passing a column view instance directly to a span parameter or variable.

Checklist

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

@davidwendt davidwendt self-assigned this Nov 6, 2025
@davidwendt davidwendt requested a review from a team as a code owner November 6, 2025 22:59
@davidwendt davidwendt added the 3 - Ready for Review Ready for review by team label Nov 6, 2025
@davidwendt davidwendt added libcudf Affects libcudf (C++/CUDA) code. improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Nov 6, 2025
Copy link
Contributor

@karthikeyann karthikeyann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.
nit: update cudf/cpp/tests/column/column_view_device_span_test.cpp too as part of this PR or new one.

Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a GTest. We need to verify that the numeric/chrono cases work, other types don't work, and nullable columns error as expected. Hopefully we have device_span tests that can be copied here.

@davidwendt
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit ccc6b63 into rapidsai:main Nov 11, 2025
261 of 263 checks passed
@davidwendt davidwendt deleted the colview-std-span-op branch November 11, 2025 15:12
trxcllnt pushed a commit to trxcllnt/cudf that referenced this pull request Nov 12, 2025
Adds a `cuda::std::span` operator to the `cudf::column_view` class to allow passing a column view instance directly to a span parameter or variable.

Authors:
  - David Wendt (https://github.com/davidwendt)

Approvers:
  - Karthikeyan (https://github.com/karthikeyann)
  - Bradley Dice (https://github.com/bdice)

URL: rapidsai#20541
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review Ready for review by team improvement Improvement / enhancement to an existing function libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants