Skip to content

Add env overloads for Select::* in place algorithms#8018

Merged
gonidelis merged 2 commits intoNVIDIA:mainfrom
gonidelis:select_inplace_env
Mar 19, 2026
Merged

Add env overloads for Select::* in place algorithms#8018
gonidelis merged 2 commits intoNVIDIA:mainfrom
gonidelis:select_inplace_env

Conversation

@gonidelis
Copy link
Copy Markdown
Member

Adds in-place DeviceSelect environment overloads

fixes #7945

@gonidelis gonidelis requested a review from a team as a code owner March 13, 2026 00:00
@gonidelis gonidelis requested a review from pauleonix March 13, 2026 00:00
@github-project-automation github-project-automation Bot moved this to Todo in CCCL Mar 13, 2026
@cccl-authenticator-app cccl-authenticator-app Bot moved this from Todo to In Review in CCCL Mar 13, 2026
@github-actions

This comment has been minimized.

REQUIRE(data == expected_output);
}

C2H_TEST("cub::DeviceSelect::If in-place accepts env with stream", "[select][env]")
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.

We dopcument:

//! - | The range [d_out, d_out + *d_num_selected_out) shall not overlap
//! | [d_in, d_in + num_items) nor d_num_selected_out in any way.

Is that actually valid? Asking for someone that needs that

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.

cub::DeviceSelect::If has two overloads, an in-place one and one with separate source and destination iterators.

Copy link
Copy Markdown
Contributor

@NaderAlAwar NaderAlAwar left a comment

Choose a reason for hiding this comment

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

Left a few comments to improve documentation

Comment thread cub/cub/device/device_select.cuh
Comment thread cub/cub/device/device_select.cuh
Comment thread cub/cub/device/device_select.cuh Outdated
@gonidelis gonidelis enabled auto-merge (squash) March 19, 2026 02:16
@github-actions
Copy link
Copy Markdown
Contributor

🥳 CI Workflow Results

🟩 Finished in 2h 20m: Pass: 100%/249 | Total: 5d 04h | Max: 2h 19m | Hits: 93%/157552

See results here.

@gonidelis gonidelis merged commit 161f870 into NVIDIA:main Mar 19, 2026
263 of 266 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in CCCL Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[FEA]: Add in-place env overloads for DeviceSelect

4 participants