fix: disallow None default value, instead use empty tuple

GazzolaLab · Jun 25, 2024 · 069e2cd · 069e2cd
1 parent 06cc1e0
commit 069e2cd
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/elastica/boundary_conditions.py b/elastica/boundary_conditions.py
@@ -40,8 +40,8 @@ class ConstraintBase(ABC, Generic[S]):
     def __init__(
         self,
         *args: Any,
-        constrained_position_idx: ConstrainingIndex,
-        constrained_director_idx: ConstrainingIndex,
+        constrained_position_idx: ConstrainingIndex = (),
+        constrained_director_idx: ConstrainingIndex = (),
         **kwargs: Any,
     ) -> None:
         """Initialize boundary condition"""

diff --git a/elastica/modules/constraints.py b/elastica/modules/constraints.py
@@ -151,8 +151,8 @@ def using(
         self,
         cls: Type[ConstraintBase],
         *args: Any,
-        constrained_position_idx: ConstrainingIndex = None,
-        constrained_director_idx: ConstrainingIndex = None,
+        constrained_position_idx: ConstrainingIndex = (),
+        constrained_director_idx: ConstrainingIndex = (),
         **kwargs: Any,
     ) -> Self:
         """

diff --git a/elastica/typing.py b/elastica/typing.py
@@ -58,7 +58,7 @@
 
 # Indexing types
 # TODO: Maybe just use slice??
-ConstrainingIndex: TypeAlias = list[int] | tuple[int] | np.typing.NDArray | None
+ConstrainingIndex: TypeAlias = list[int] | tuple[int, ...] | np.typing.NDArray
 ConnectionIndex: TypeAlias = (
     int | np.int_ | list[int] | tuple[int] | np.typing.NDArray | None
 )