-
Notifications
You must be signed in to change notification settings - Fork 297
Deprecate Device.set_current()? #548
Copy link
Copy link
Open
Labels
RFCPlans and announcementsPlans and announcementscuda.coreEverything related to the cuda.core moduleEverything related to the cuda.core moduleenhancementAny code-related improvementsAny code-related improvements
Milestone
Metadata
Metadata
Assignees
Labels
RFCPlans and announcementsPlans and announcementscuda.coreEverything related to the cuda.core moduleEverything related to the cuda.core moduleenhancementAny code-related improvementsAny code-related improvements
Type
Fields
Give feedbackNo fields configured for Enhancement.
I forgot to open an issue to track an offline discussion I had with @pciolkosz...
Currently,
cuda.corediverges withcudaxin a few places, and this is one of them.cuda.corefocuses 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.