Skip to content

Use cuda::stream_ref for compute APIs - #23649

Open
vyasr wants to merge 5 commits into
NVIDIA:mainfrom
vyasr:codex/cuda-stream-ref-batch-3
Open

Use cuda::stream_ref for compute APIs#23649
vyasr wants to merge 5 commits into
NVIDIA:mainfrom
vyasr:codex/cuda-stream-ref-batch-3

Conversation

@vyasr

@vyasr vyasr commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Description

This third batch migrates compute libcudf joins, groupby, reductions, rolling, sorting, transform, unary, and binary operation APIs and tests from rmm::cuda_stream_view to cuda::stream_ref. It also includes small follow-up documentation fixes from the previous stream-ref batch.

Contributes to #23636

Checklist

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

@vyasr
vyasr requested a review from a team as a code owner August 13, 2026 17:09
@vyasr
vyasr requested review from abigalekim and lamarrr August 13, 2026 17:09
@vyasr vyasr added 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 labels Aug 13, 2026 — with ChatGPT Codex Connector
Comment thread cpp/tests/join/join_tests.cpp Outdated
@@ -2431,7 +2432,7 @@ TEST_F(JoinTest, HashJoinInnerMatchContext)

auto const host_match_counts =
cudf::detail::make_host_vector_async(*match_context._match_counts, stream);
stream.synchronize();
stream.wait();

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please use .sync() instead. xref: #23653

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.

2 participants