Skip to content

Expose stream parameter in public reduction APIs - #15737

Merged
rapids-bot[bot] merged 3 commits into
NVIDIA:branch-24.06from
srinivasyadav18:expose_stream_reduction
May 16, 2024
Merged

Expose stream parameter in public reduction APIs#15737
rapids-bot[bot] merged 3 commits into
NVIDIA:branch-24.06from
srinivasyadav18:expose_stream_reduction

Conversation

@srinivasyadav18

Copy link
Copy Markdown
Contributor

Description

Add stream parameter to public reduction APIs:

  • reduce()
  • segmented_reduce()
  • scan()
  • minmax()

Reference #13744

Checklist

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

@srinivasyadav18
srinivasyadav18 requested a review from a team as a code owner May 13, 2024 21:03
@copy-pr-bot

copy-pr-bot Bot commented May 13, 2024

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions Bot added libcudf Affects libcudf (C++/CUDA) code. CMake CMake build issue labels May 13, 2024
@davidwendt davidwendt added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change 3 - Ready for Review Ready for review by team labels May 13, 2024
@davidwendt

Copy link
Copy Markdown
Contributor

/ok to test

@mhaseeb123 mhaseeb123 left a comment

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.

LGTM!

@srinivasyadav18

Copy link
Copy Markdown
Contributor Author

Hello,
3 tests are still failing. I am looking into it ;-). Any leads would be appreciated. Thanks!

Comment thread cpp/tests/streams/reduction_test.cpp Outdated
Comment thread cpp/tests/streams/reduction_test.cpp Outdated
@srinivasyadav18

Copy link
Copy Markdown
Contributor Author

Thank you @davidwendt. Your suggestions worked out!

@davidwendt

Copy link
Copy Markdown
Contributor

/ok to test

@vuule vuule added 5 - Ready to Merge Testing and reviews complete, ready to merge and removed 3 - Ready for Review Ready for review by team labels May 14, 2024
@vuule

vuule commented May 15, 2024

Copy link
Copy Markdown
Collaborator

/ok to test

@PointKernel PointKernel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@srinivasyadav18 Welcome and thanks for the contribution!

Just FYI, CI is broken right now thus it will take a bit longer to merge the current PR.

@PointKernel

Copy link
Copy Markdown
Member

/merge

@rapids-bot
rapids-bot Bot merged commit ec07927 into NVIDIA:branch-24.06 May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to Merge Testing and reviews complete, ready to merge CMake CMake build issue 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.

5 participants