cuda.core: expose named resource state without changing truthiness

Replace lifecycle-dependent truthiness with explicit is_closed and is_valid properties while preserving the historical truth value of cuda.core objects.

Author: Andy-Jost

cuda_core/cuda/core/_context.pyi

@@ -32,8 +32,9 @@ class Context:
     def _handle(self) -> cuda.bindings.driver.CUcontext | None:
         ...
 
-    def __bool__(self) -> bool:
-        ...
+    @property
+    def is_closed(self) -> bool:
+        """Whether this context has been closed."""
 
     @property
     def is_green(self) -> bool:

cuda_core/cuda/core/_context.pyx

@@ -74,8 +74,10 @@ cdef class Context:
     def _handle(self) -> cuda.bindings.driver.CUcontext | None:
         return self.handle
 
-    def __bool__(self) -> bool:
-        return self._h_context.get() != NULL
+    @property
+    def is_closed(self) -> bool:
+        """Whether this context has been closed."""
+        return self._h_context.get() == NULL
 
     @property
     def is_green(self) -> bool:

cuda_core/cuda/core/_device.pyx

@@ -1278,7 +1278,7 @@ class Device:
         if ctx is not None:
             # TODO: revisit once Context is cythonized
             assert_type(ctx, Context)
-            if not ctx:
+            if ctx.is_closed:
                 raise RuntimeError("Context has been closed")
             if ctx._device_id != self._device_id:
                 raise RuntimeError(

cuda_core/cuda/core/_event.pyi

@@ -72,8 +72,9 @@ class Event:
     def __init__(self, *args, **kwargs) -> None:
         ...
 
-    def __bool__(self) -> bool:
-        ...
+    @property
+    def is_closed(self) -> bool:
+        """Whether this event has been closed."""
 
     def __isub__(self, other: object):
         ...

cuda_core/cuda/core/_event.pyx

@@ -157,8 +157,10 @@ cdef class Event:
         """
         self._h_event.reset()
 
-    def __bool__(self) -> bool:
-        return self._h_event.get() != NULL
+    @property
+    def is_closed(self) -> bool:
+        """Whether this event has been closed."""
+        return self._h_event.get() == NULL
 
     def __isub__(self, other: object):
         return NotImplemented

cuda_core/cuda/core/_graphics.pyi

@@ -213,8 +213,9 @@ class GraphicsResource:
     def handle(self) -> int:
         """The raw ``CUgraphicsResource`` handle as a Python int."""
 
-    def __bool__(self) -> bool:
-        ...
+    @property
+    def is_closed(self) -> bool:
+        """Whether this graphics resource has been closed."""
 
     @property
     def resource_handle(self) -> int:

cuda_core/cuda/core/_graphics.pyx

@@ -347,8 +347,10 @@ cdef class GraphicsResource:
         """The raw ``CUgraphicsResource`` handle as a Python int."""
         return as_intptr(self._handle)
 
-    def __bool__(self) -> bool:
-        return self._handle.get() != NULL
+    @property
+    def is_closed(self) -> bool:
+        """Whether this graphics resource has been closed."""
+        return self._handle.get() == NULL
 
     @property
     def resource_handle(self) -> int:

cuda_core/cuda/core/_kernel_arg_handler.pyx

@@ -282,7 +282,7 @@ cdef class ParamHolder:
         for i, arg in enumerate(kernel_args):
             arg_type = type(arg)
             if arg_type is Buffer:
-                if not arg:
+                if arg.is_closed:
                     raise RuntimeError("Buffer has been closed")
                 # we need the address of where the actual buffer address is stored
                 if type(arg.handle) is int:
@@ -329,7 +329,7 @@ cdef class ParamHolder:
                     continue
                 # If no exact types are found, fallback to slower `isinstance` check
                 elif isinstance(arg, Buffer):
-                    if not arg:
+                    if arg.is_closed:
                         raise RuntimeError("Buffer has been closed")
                     if isinstance(arg.handle, int):
                         prepare_arg[intptr_t](self.data, self.data_addresses, arg.handle, i)

cuda_core/cuda/core/_linker.pyi

@@ -35,8 +35,9 @@ class Linker:
     def __init__(self, options: LinkerOptions | None=None, *object_codes: ObjectCode):
         ...
 
-    def __bool__(self) -> bool:
-        ...
+    @property
+    def is_closed(self) -> bool:
+        """Whether this linker has been closed."""
 
     def link(self, target_type: ObjectCodeFormatType | str) -> ObjectCode:
         """Link the provided object codes into a single output of the specified target type.

cuda_core/cuda/core/_linker.pyx

@@ -81,10 +81,12 @@ cdef class Linker:
     def __init__(self, *object_codes: ObjectCode, options: LinkerOptions | None = None):
         Linker_init(self, object_codes, options)
 
-    def __bool__(self) -> bool:
+    @property
+    def is_closed(self) -> bool:
+        """Whether this linker has been closed."""
         if self._use_nvjitlink:
-            return self._nvjitlink_handle.get() != NULL
-        return self._culink_handle.get() != NULL
+            return self._nvjitlink_handle.get() == NULL
+        return self._culink_handle.get() == NULL
 
     def link(self, target_type: ObjectCodeFormatType | str) -> ObjectCode:
         """Link the provided object codes into a single output of the specified target type.
@@ -104,7 +106,7 @@ cdef class Linker:
             Ensure that input object codes were compiled with appropriate
             flags for linking (e.g., relocatable device code enabled).
         """
-        if not self:
+        if self.is_closed:
             raise RuntimeError("Linker has been closed")
         return Linker_link(self, str(target_type))
 
@@ -119,7 +121,7 @@ cdef class Linker:
         # After link(), the decoded log is cached here.
         if self._error_log is not None:
             return self._error_log
-        if not self:
+        if self.is_closed:
             raise RuntimeError("Linker has been closed")
         cdef cynvjitlink.nvJitLinkHandle c_h
         cdef size_t c_log_size = 0
@@ -147,7 +149,7 @@ cdef class Linker:
         # After link(), the decoded log is cached here.
         if self._info_log is not None:
             return self._info_log
-        if not self:
+        if self.is_closed:
             raise RuntimeError("Linker has been closed")
         cdef cynvjitlink.nvJitLinkHandle c_h
         cdef size_t c_log_size = 0