Merge pull request #267 from ajhynes7/fix_warnings

Fix warnings

HISTORY.rst

@@ -2,6 +2,15 @@
 History
 =======
 
+6.0.1 (2021-03-25)
+------------------
+
+Fixes
+~~~~~
+* Wrap `filterwarnings("error")` in a `catch_warnings` context manager, in `__BaseArray.__new__()`.
+  Now the warning level is reset at the end of the context manager.
+
+
 6.0.0 (2021-03-21)
 ------------------
 

pyproject.toml

@@ -1,7 +1,7 @@
 [tool.poetry]
 
 name = "scikit-spatial"
-version = "6.0.0"
+version = "6.0.1"
 description = "Spatial objects and computations based on NumPy arrays."
 license = "BSD"
 

src/skspatial/__init__.py

@@ -2,4 +2,4 @@
 
 __author__ = "Andrew Hynes"
 __email__ = 'andrewjhynes@gmail.com'
-__version__ = '6.0.0'
+__version__ = '6.0.1'

src/skspatial/objects/_base_array.py

@@ -23,14 +23,16 @@ class _BaseArray(np.ndarray, _BaseSpatial):
 
     def __new__(cls: Type[Array], array: array_like) -> Array:
 
-        try:
+        with warnings.catch_warnings():
+
             warnings.filterwarnings("error")
-            np.array(array)
 
-        except np.VisibleDeprecationWarning as error:
+            try:
+                np.array(array)
 
-            if str(error).startswith("Creating an ndarray from ragged nested sequences"):
-                raise ValueError("The array must not contain sequences with different lengths.")
+            except np.VisibleDeprecationWarning as error:
+                if str(error).startswith("Creating an ndarray from ragged nested sequences"):
+                    raise ValueError("The array must not contain sequences with different lengths.")
 
         if np.size(array) == 0:
             raise ValueError("The array must not be empty.")