feat: automatically convert bounds list to tuple in constructor

Author: Elan456

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "fastquadtree"
-version = "1.4.0"
+version = "1.4.1"
 edition = "2021"
 
 [lib]

Cargo.toml

@@ -91,11 +91,20 @@ def __init__(
         track_objects: bool = False,
         dtype: str = "f32",
     ):
+        # Handle some bounds validation and list --> tuple conversion
+        if type(bounds) is not tuple:
+            bounds = tuple(bounds)  # pyright: ignore[reportAssignmentType]
+        if len(bounds) != 4:
+            raise ValueError(
+                "bounds must be a tuple of four numeric values (x min, y min, x max, y max)"
+            )
+
         self._bounds = bounds
+
         self._max_depth = max_depth
         self._capacity = capacity
         self._dtype = dtype
-        self._native = self._new_native(bounds, capacity, max_depth)
+        self._native = self._new_native(self._bounds, self._capacity, self._max_depth)
 
         self._track_objects = bool(track_objects)
         self._store: ObjStore[ItemType] | None = ObjStore() if track_objects else None

pysrc/fastquadtree/_base_quadtree.py

@@ -163,3 +163,17 @@ def test_insert_many_exception_for_out_of_bounds():
 
     with pytest.raises(ValueError):
         qt.insert_many(points)
+
+
+def test_construct_quadtree_with_list_bounds():
+    bounds_list = [0.0, 0.0, 500.0, 500.0]
+    qt = QuadTree(bounds_list, capacity=4)  # pyright: ignore[reportArgumentType]
+    assert qt._bounds == (0.0, 0.0, 500.0, 500.0)
+
+
+def test_construct_quadtree_with_invalid_bounds_length():
+    bounds_invalid = (0.0, 0.0, 500.0)  # Only three values instead of four
+    with pytest.raises(
+        ValueError, match="bounds must be a tuple of four numeric values"
+    ):
+        QuadTree(bounds_invalid, capacity=4)  # pyright: ignore[reportArgumentType]