Skip to content

Support non-copyable stream types in DeviceTransform#7915

Open
bernhardmgruber wants to merge 1 commit intoNVIDIA:mainfrom
bernhardmgruber:fix_stream_overload
Open

Support non-copyable stream types in DeviceTransform#7915
bernhardmgruber wants to merge 1 commit intoNVIDIA:mainfrom
bernhardmgruber:fix_stream_overload

Conversation

@bernhardmgruber
Copy link
Contributor

This is a fix for another bug introduced in #6204

This is a fix for another bug introduced in NVIDIA#6204
@bernhardmgruber bernhardmgruber requested a review from a team as a code owner March 6, 2026 11:34
@github-project-automation github-project-automation bot moved this to Todo in CCCL Mar 6, 2026
@bernhardmgruber bernhardmgruber changed the title Support non-copyable stream types in DeviceTransform Support non-copyable stream types in DeviceTransform Mar 6, 2026
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Mar 6, 2026
::cuda::std::tuple<RandomAccessIteratorsOut...> outputs,
NumItemsT num_items,
TransformOp transform_op,
Env env = {})
Copy link
Contributor

Choose a reason for hiding this comment

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

Strongly wondering whether it wouldnt be better to have an overload that forwards, but we should definitely make a definite decision on what we expect of environements

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think taking the environment by && would work as well here, yes. We should decide this.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

🥳 CI Workflow Results

🟩 Finished in 5h 21m: Pass: 100%/249 | Total: 9d 01h | Max: 4h 00m | Hits: 73%/155156

See results here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants