use typeguard>=4.2, fixup some type checks

agronholm/typeguard#442 was fixed in typeguard 4.2.0

apis/python/requirements_dev.txt

@@ -3,5 +3,5 @@ ruff
 pytest
 pytest-cov
 sparse
-typeguard==4.1.5
+typeguard==4.2.1
 types-setuptools

apis/python/src/tiledbsoma/_collection.py

@@ -185,7 +185,7 @@ def add_new_collection(
     def add_new_collection(
         self,
         key: str,
-        kind: Optional[Type[AnyTileDBCollection]] = None,
+        kind: Optional[Type[CollectionBase]] = None,  # type: ignore[type-arg]
         *,
         uri: Optional[str] = None,
         platform_config: Optional[options.PlatformConfig] = None,
@@ -377,7 +377,6 @@ def add_new_sparse_ndarray(self, key: str, **kwargs: Any) -> SparseNDArray:
         """
         return self._add_new_ndarray(SparseNDArray, key, **kwargs)
 
-    @typeguard_ignore
     def _add_new_element(
         self,
         key: str,
@@ -444,7 +443,7 @@ def __getitem__(self, key: str) -> CollectionElementType:
                     entry.entry.wrapper_type.open, uri, mode, context, timestamp
                 )
             # Since we just opened this object, we own it and should close it.
-            self._close_stack.enter_context(entry.soma)  # type: ignore[arg-type]
+            self._close_stack.enter_context(entry.soma)
         return cast(CollectionElementType, entry.soma)
 
     def set(

apis/python/src/tiledbsoma/_factory.py

@@ -122,7 +122,7 @@ def open(
         Experimental.
     """
     context = _validate_soma_tiledb_context(context)
-    obj: TileDBObject[_Wrapper] = _open_internal(  # type: ignore[no-untyped-call,valid-type]
+    obj: TileDBObject[_Wrapper] = _open_internal(  # type: ignore[valid-type]
         _tdb_handles.open, uri, mode, context, tiledb_timestamp
     )
     try:
@@ -143,7 +143,6 @@ def open(
         raise
 
 
-@no_type_check
 def _open_internal(
     opener: Callable[
         [str, options.OpenMode, SOMATileDBContext, Optional[OpenTimestamp]], _Wrapper

apis/python/src/tiledbsoma/options/_soma_tiledb_context.py

@@ -7,8 +7,8 @@
 import functools
 import threading
 import time
-from concurrent import futures
-from typing import Any, Dict, Mapping, Optional, Union
+from concurrent.futures import ThreadPoolExecutor
+from typing import Any, Dict, Mapping, Optional, Type, Union, cast
 
 import tiledb
 from somacore import ContextBase
@@ -46,8 +46,10 @@ def _maybe_timestamp_ms(input: Optional[OpenTimestamp]) -> Optional[int]:
     return to_timestamp_ms(input)
 
 
-_SENTINEL = object()
-"""Sentinel object to distinguish default parameters from None."""
+class _SENTINEL:
+    """Sentinel object to distinguish default parameters from None."""
+
+    pass
 
 
 class SOMATileDBContext(ContextBase):
@@ -68,7 +70,7 @@ def __init__(
         tiledb_ctx: Optional[tiledb.Ctx] = None,
         tiledb_config: Optional[Dict[str, Union[str, float]]] = None,
         timestamp: Optional[OpenTimestamp] = None,
-        threadpool: Optional[futures.ThreadPoolExecutor] = None,
+        threadpool: Optional[ThreadPoolExecutor] = None,
     ) -> None:
         """Initializes a new SOMATileDBContext.
 
@@ -138,7 +140,7 @@ def __init__(
         """The TileDB context to use, either provided or lazily constructed."""
         self._timestamp_ms = _maybe_timestamp_ms(timestamp)
 
-        self.threadpool = threadpool or futures.ThreadPoolExecutor()
+        self.threadpool = threadpool or ThreadPoolExecutor()
         """User specified threadpool. If None, we'll instantiate one ourselves."""
         self._native_context: Optional[clib.SOMAContext] = None
         """Lazily construct clib.SOMAContext."""
@@ -219,8 +221,8 @@ def replace(
         *,
         tiledb_config: Optional[Dict[str, Any]] = None,
         tiledb_ctx: Optional[tiledb.Ctx] = None,
-        timestamp: Optional[OpenTimestamp] = _SENTINEL,  # type: ignore[assignment]
-        threadpool: Optional[futures.ThreadPoolExecutor] = _SENTINEL,  # type: ignore[assignment]
+        timestamp: Union[None, OpenTimestamp, Type[_SENTINEL]] = _SENTINEL,
+        threadpool: Union[None, ThreadPoolExecutor, Type[_SENTINEL]] = _SENTINEL,
     ) -> Self:
         """Create a copy of the context, merging changes.
 
@@ -259,17 +261,19 @@ def replace(
                 new_config = self._internal_tiledb_config()
                 new_config.update(tiledb_config)
                 tiledb_config = {k: v for (k, v) in new_config.items() if v is not None}
+
             if timestamp is _SENTINEL:
                 # Keep the existing timestamp if not overridden.
                 timestamp = self._timestamp_ms
             if threadpool is _SENTINEL:
                 # Keep the existing threadpool if not overridden.
                 threadpool = self.threadpool
+
         return type(self)(
             tiledb_config=tiledb_config,
             tiledb_ctx=tiledb_ctx,
-            timestamp=timestamp,
-            threadpool=threadpool,
+            timestamp=cast(Optional[OpenTimestamp], timestamp),
+            threadpool=cast(Optional[ThreadPoolExecutor], threadpool),
         )
 
     def _open_timestamp_ms(self, in_timestamp: Optional[OpenTimestamp]) -> int: