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.

Sign up for GitHub

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

Jump to bottom

[FEA] Deprecate rmm.pylibrmm.cuda_stream and move associated code to rmm.pylibrmm.stream #1782

Open
Matt711 opened this issue Jan 9, 2025 · 0 comments
Open

[FEA] Deprecate rmm.pylibrmm.cuda_stream and move associated code to rmm.pylibrmm.stream #1782

Matt711 opened this issue Jan 9, 2025 · 0 comments
Assignees
@Matt711
Labels
? - Needs Triage Need team to review and classify feature request New feature or request

Comments

@Matt711
Copy link
Contributor

Matt711 commented Jan 9, 2025 •
edited
Loading

Is your feature request related to a problem? Please describe.
RMM should have a single public stream module. Eg.

from rmm.pylibrmm.stream import Stream, CudaStream, ...

Describe the solution you'd like
At a minimum, we should deprecate the public cuda_stream module and move the CudaStream calss to the public stream module (created in #1775).

Describe alternatives you've considered
We might be able to remove the CudaStream class completely and move all functionality to the main public Stream class.

Additional context
Depends on #1770
@Matt711 Matt711 added ? - Needs Triage Need team to review and classify feature request New feature or request labels Jan 9, 2025
@Matt711 Matt711 self-assigned this Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Matt711 Matt711

Labels
? - Needs Triage Need team to review and classify feature request New feature or request
Projects
RMM Project Board
Status: To-do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Matt711