Remove boltons dependencies

Author: krafczyk

setup.cfg

@@ -32,7 +32,6 @@ keywords =
 package_dir =
     = src
 install_requires =
-    boltons
     dill
     tqdm
     numpy

src/dryml/core2/definition.py

@@ -135,7 +135,7 @@ def skip_args(self) -> bool:
         else:
             return False
 
-    # --- mapping interface (for get_definition_view / remap compatibility) ---
+    # --- mapping interface (for view compatibility) ---
     def __getitem__(self, k: str) -> Any:
         if k == "cls":
             if self._cls is None:

src/dryml/core2/freeze.py

@@ -42,8 +42,7 @@ def __repr__(self) -> str:
 
 class FrozenDict(ABCMapping):
     """
-    Immutable mapping, not a dict subclass (important: boltons.remap default_enter
-    will build a plain dict for dictlikes, avoiding 'can’t populate an immutable dict').
+    Immutable mapping, not a dict subclass
     """
     __slots__ = ("_items", "_dict")
 

src/dryml/core2/repo.py

@@ -776,33 +776,6 @@ def _add_object(obj: Any):
         for arg in args:
             _add_object(arg)
 
-
-    #     def _enter(path, key, value):
-    #         if isinstance(value, Object):
-    #             return {}, get_object_view(value)
-    #         elif isinstance(value, ConcreteDefinition):
-    #             return {}, get_definition_view(value)
-    #         else:
-    #             return default_enter(path, key, value)
-
-    #     def _exit(path, key, value, new_parent, new_items):
-    #         if isinstance(value, Object):
-    #             _add_object(value)
-    #             return value
-    #         elif isinstance(value, ConcreteDefinition):
-    #             if value._obj is None:
-    #                 raise ValueError("Can't use a ConcreteDefinition without _obj pointer..")
-    #             _add_object(value._obj)
-    #             return value._obj
-    #         else:
-    #             return default_exit(path, key, value, new_parent, new_items)
-
-    #     remap(
-    #          args,
-    #          enter=_enter,
-    #          visit=default_visit,
-    #          exit=_exit)
-
     def flush(self):
         # Commit all stores
         for store in self.stores:

src/dryml/core2/utils/general.py

@@ -8,7 +8,7 @@
 import io
 import glob
 from typing import Optional, Callable
-from collections.abc import Mapping, ItemsView
+from collections.abc import Mapping, Iterable, ItemsView
 from inspect import getmodule, isclass, \
     Parameter, signature
 
@@ -85,6 +85,26 @@ def is_dictlike(val):
     return isinstance(val, Mapping)
 
 
+def is_collection(val) -> bool:
+    """
+    True for non-mapping iterables (lists, tuples, sets, generators, etc.),
+    but False for string/bytes-like objects and mappings.
+    """
+    if val is None:
+        return False
+
+    # Treat these as scalars/leaves, not collections
+    if isinstance(val, (str, bytes, bytearray, memoryview)):
+        return False
+
+    # Leave mappings to is_dictlike()
+    if isinstance(val, Mapping):
+        return False
+
+    # Iterable => collection
+    return isinstance(val, Iterable)
+
+
 def is_stream(obj) -> bool:
     return isinstance(obj, io.IOBase)
 

src/dryml/core2/utils/stable_hash.py

@@ -9,9 +9,8 @@
 
 import numpy as np
 
-from .general import is_dictlike
+from .general import is_dictlike, is_collection
 from .recurse import cycle_detect
-from boltons.iterutils import remap, is_collection, default_enter
 
 def stable_int_hash(s: str, *, bits: int = 64) -> int:
     # blake2b is fast and stable; digest_size controls output size
@@ -121,7 +120,6 @@ def stable_hash_value(value) -> str:
 
 def _is_container(value) -> bool:
     """
-    Container in the sense of remap traversal.
     Note: np.ndarray is explicitly *not* treated as a container here.
     """
     return (is_dictlike(value) or is_collection(value)) and not isinstance(value, np.ndarray)