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

Use stream_allocator_adaptor constructor instead of factory. #4557

Merged
merged 7 commits into from
Aug 5, 2024

Conversation

bdice
Copy link
Contributor

@bdice bdice commented Jul 26, 2024

This PR uses the stream_allocator_adaptor constructor instead of a factory. With CTAD, we do not need the factory and can use the constructor directly. The factory will be deprecated in rapidsai/rmm#1626.

@bdice bdice requested review from a team as code owners July 26, 2024 23:33
@bdice bdice requested a review from jameslamb July 26, 2024 23:33
@bdice bdice added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 26, 2024
@bdice bdice self-assigned this Jul 26, 2024
Copy link
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

It looks like most of the diff here (and packaging-codeowners being added as a reviewer) is coming from the fact that this has branch-24.10 commits on it but is targeting branch-24.08. I suspect this was meant to target branch-24.10.

@bdice bdice changed the base branch from branch-24.08 to branch-24.10 July 29, 2024 14:08
@bdice
Copy link
Contributor Author

bdice commented Jul 29, 2024

@jameslamb Fixed! Thanks, just forgot to update the base branch when opening.

@bdice bdice requested review from jameslamb and removed request for a team and jameslamb July 29, 2024 14:08
@bdice bdice dismissed jameslamb’s stale review July 29, 2024 14:09

CI/packaging reviews not needed after base branch fix

@jameslamb jameslamb self-requested a review July 29, 2024 14:49
Copy link
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

Ok yeah I figured, thanks. Approved from me (not that you need it any more).

@bdice
Copy link
Contributor Author

bdice commented Aug 5, 2024

/merge

@rapids-bot rapids-bot bot merged commit 8f86c82 into rapidsai:branch-24.10 Aug 5, 2024
137 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuGraph improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants