-
Notifications
You must be signed in to change notification settings - Fork 982
Add cuda::std::span operator to cudf::column_view #20541
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
karthikeyann
left a comment
There was a problem hiding this 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.
There was a problem hiding this 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.
|
/merge |
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
Description
Adds a
cuda::std::spanoperator to thecudf::column_viewclass to allow passing a column view instance directly to a span parameter or variable.Checklist