Skip to content

Commit 55d4516

Browse files
leofangclaude
andcommitted
Restrict ContextOptions.resources to Sequence only
DeviceResourcesT no longer accepts a bare SMResource or WorkqueueResource — only Sequence[SMResource | WorkqueueResource]. Remove the auto-wrapping isinstance check in Device.create_context(). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 7c12d84 commit 55d4516

2 files changed

Lines changed: 2 additions & 6 deletions

File tree

cuda_core/cuda/core/_context.pyx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ from cuda.core._utils.cuda_utils cimport HANDLE_RETURN
2525
__all__ = ['Context', 'ContextOptions']
2626

2727

28-
DeviceResourcesT = SMResource | WorkqueueResource | Sequence[SMResource | WorkqueueResource]
28+
DeviceResourcesT = Sequence[SMResource | WorkqueueResource]
2929

3030

3131
cdef class Context:

cuda_core/cuda/core/_device.pyx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1308,11 +1308,7 @@ class Device:
13081308
if options.resources is None:
13091309
raise NotImplementedError("WIP: https://github.com/NVIDIA/cuda-python/issues/189")
13101310

1311-
resources = options.resources
1312-
if isinstance(resources, (SMResource, WorkqueueResource)):
1313-
resources = (resources,)
1314-
else:
1315-
resources = tuple(resources)
1311+
resources = tuple(options.resources)
13161312
if len(resources) == 0:
13171313
raise ValueError("ContextOptions.resources must not be empty")
13181314

0 commit comments

Comments
 (0)