Skip to content
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

Update cub 1.13.1 calls #4334

Closed
wants to merge 1 commit into from

Conversation

dantegd
Copy link
Member

@dantegd dantegd commented Nov 8, 2021

Updates calls to account for API changes in CUB 1.13.1 allowing upgrade to thrust 1.13+

@dantegd dantegd added 5 - DO NOT MERGE Hold off on merging; see PR for details improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Nov 8, 2021
@dantegd dantegd requested a review from a team as a code owner November 8, 2021 19:11
@github-actions
Copy link

github-actions bot commented Dec 8, 2021

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.

@dantegd dantegd changed the base branch from branch-21.12 to branch-22.02 January 7, 2022 00:17
@dantegd dantegd closed this Jan 11, 2022
rapids-bot bot pushed a commit that referenced this pull request Jan 12, 2022
The following changes are required for compilation with Thrust 1.15, which is becoming the minimum required version across all of RAPIDS for 22.02.

The changes needed:
- Removal of global `using namespace std`. This makes calls to `cub::` ambigous as now thrust::cub ( new since 1.14? ) and cub both are valid.
- Updates from #4334, which fixes `cub::DeviceSegmentedReduce` signature changes

Authors:
  - Robert Maynard (https://github.com/robertmaynard)

Approvers:
  - Dante Gama Dessavre (https://github.com/dantegd)

URL: #4469
vimarsh6739 pushed a commit to vimarsh6739/cuml that referenced this pull request Oct 9, 2023
The following changes are required for compilation with Thrust 1.15, which is becoming the minimum required version across all of RAPIDS for 22.02.

The changes needed:
- Removal of global `using namespace std`. This makes calls to `cub::` ambigous as now thrust::cub ( new since 1.14? ) and cub both are valid.
- Updates from rapidsai#4334, which fixes `cub::DeviceSegmentedReduce` signature changes

Authors:
  - Robert Maynard (https://github.com/robertmaynard)

Approvers:
  - Dante Gama Dessavre (https://github.com/dantegd)

URL: rapidsai#4469
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - DO NOT MERGE Hold off on merging; see PR for details CUDA/C++ improvement Improvement / enhancement to an existing function inactive-30d non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant