More mypy improvements

pyproject.toml

@@ -124,6 +124,7 @@ plugins = [
 ]
 strict = true
 warn_return_any = false
+warn_unreachable = true
 
 [tool.pytest.ini_options]
 addopts = [

src/sleplet/functions/slepian.py

@@ -82,9 +82,6 @@ def _check_rank(
         cls,  # noqa: ANN101
         v: int,
     ) -> int:
-        if not isinstance(v, int):
-            msg = "rank should be an integer"
-            raise TypeError(msg)
         if v < 0:
             msg = "rank cannot be negative"
             raise ValueError(msg)

src/sleplet/functions/spherical_harmonic.py

@@ -55,9 +55,6 @@ def _check_ell(
         v: int,
         info: pydantic.ValidationInfo,
     ) -> int:
-        if not isinstance(v, int):
-            msg = "ell should be an integer"
-            raise TypeError(msg)
         if v < 0:
             msg = "ell should be positive"
             raise ValueError(msg)
@@ -72,9 +69,6 @@ def _check_m(
         v: int,
         info: pydantic.ValidationInfo,
     ) -> int:
-        if not isinstance(v, int):
-            msg = "m should be an integer"
-            raise TypeError(msg)
         if abs(v) > info.data["ell"]:
             msg = "the magnitude of m should be less than ell"
             raise ValueError(msg)

src/sleplet/meshes/_mesh_slepian_decomposition.py

@@ -117,9 +117,6 @@ def _detect_method(self: typing_extensions.Self) -> None:
 
     def _validate_rank(self: typing_extensions.Self, rank: int) -> None:
         """Check the requested rank is valid."""
-        if not isinstance(rank, int):
-            msg = "rank should be an integer"
-            raise TypeError(msg)
         if rank < 0:
             msg = "rank cannot be negative"
             raise ValueError(msg)

src/sleplet/meshes/mesh_basis_functions.py

@@ -69,9 +69,6 @@ def _check_rank(
         cls,  # noqa: ANN101
         v: int,
     ) -> int:
-        if not isinstance(v, int):
-            msg = "rank should be an integer"
-            raise TypeError(msg)
         if v < 0:
             msg = "rank cannot be negative"
             raise ValueError(msg)

src/sleplet/meshes/mesh_slepian_functions.py

@@ -71,9 +71,6 @@ def _check_rank(
         cls,  # noqa: ANN101
         v: int,
     ) -> int:
-        if not isinstance(v, int):
-            msg = "rank should be an integer"
-            raise TypeError(msg)
         if v < 0:
             msg = "rank cannot be negative"
             raise ValueError(msg)

src/sleplet/slepian/_slepian_decomposition.py

@@ -123,9 +123,6 @@ def _detect_method(self: typing_extensions.Self) -> None:
 
     def _validate_rank(self: typing_extensions.Self, rank: int) -> None:
         """Check the requested rank is valid."""
-        if not isinstance(rank, int):
-            msg = "rank should be an integer"
-            raise TypeError(msg)
         if rank < 0:
             msg = "rank cannot be negative"
             raise ValueError(msg)