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

Deprecate Device.set_current()? #548

Open
leofang opened this issue Apr 5, 2025 · 0 comments
Open

Deprecate Device.set_current()? #548

leofang opened this issue Apr 5, 2025 · 0 comments
Labels
cuda.core Everything related to the cuda.core module enhancement Any code-related improvements RFC Plans and announcements triage Needs the team's attention

Comments

@leofang
Copy link
Member

leofang commented Apr 5, 2025

I forgot to open an issue to track an offline discussion I had with @pciolkosz...

Currently, cuda.core diverges with cudax in a few places, and this is one of them. cuda.core focuses on enabling integration with existing Python projects, and having the cudart-like .set_current() method fills in the gap. However, the two projects also share many design principles, one of which is to favor explicitness, and therefore ideally we do not want any implicit/global states.

This issue tracks if there is any design opportunity to move away from the global device state, without breaking existing libraries. If such design exists, we should raise a deprecation warning asap while working toward the new design.

@leofang leofang added cuda.core Everything related to the cuda.core module enhancement Any code-related improvements RFC Plans and announcements triage Needs the team's attention labels Apr 5, 2025
@leofang leofang modified the milestone: cuda.core parking lot Apr 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuda.core Everything related to the cuda.core module enhancement Any code-related improvements RFC Plans and announcements triage Needs the team's attention
Projects
None yet
Development

No branches or pull requests

1 participant