bump ruff version in precommit (#474)

* bump ruff version in pre-commit

* reup ruff formatting

* forgot to update newest ruff

Author: jGaboardi

.pre-commit-config.yaml

@@ -1,7 +1,7 @@
 files: "spopt\/"
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.8.6"
+    rev: "v0.11.4"
     hooks:
       - id: ruff
       - id: ruff-format

spopt/region/components.py

@@ -33,7 +33,7 @@ def is_component(w, ids):
     """
 
     components = 0
-    marks = {node: 0 for node in ids}
+    marks = dict.fromkeys(ids, 0)
     q = []
     for node in ids:
         if marks[node] == 0:

spopt/region/spenclib/abstracts.py

@@ -427,9 +427,9 @@ def _spectral_bipartition(
             left_cut[left_cut] *= new_cut
             right_cut = this_cut.copy()
             right_cut[right_cut] *= ~new_cut
-            assert (
-                len(this_cut) == len(left_cut) == len(right_cut)
-            ), "Indexing Error in cutting!"
+            assert len(this_cut) == len(left_cut) == len(right_cut), (
+                "Indexing Error in cutting!"
+            )
             if ((left_cut * floor_weights).sum() > floor) & (
                 (right_cut * floor_weights).sum() > floor
             ) and (tuple(left_cut) not in accepted_cuts) & (

spopt/region/spenclib/utils.py

@@ -9,9 +9,9 @@
 
 def check_weights(W, X=None):
     if X is not None:
-        assert (
-            W.shape[0] == X.shape[0]
-        ), "W does not have the same number of samples as X"
+        assert W.shape[0] == X.shape[0], (
+            "W does not have the same number of samples as X"
+        )
     graph = sp.csc_matrix(W)
     graph.eliminate_zeros()
     components, labels = csg.connected_components(graph)

spopt/region/util.py

@@ -674,7 +674,7 @@ def generate_initial_sol(adj, n_regions):
             f"or equal to the number of areas ({n_areas})."
         )
     if n_regions == 1:
-        yield {area: 0 for area in range(n_areas)}
+        yield dict.fromkeys(range(n_areas), 0)
         return
 
     n_comps, comp_labels = csg.connected_components(adj)

spopt/tests/conftest.py

@@ -181,9 +181,9 @@ def toy_fac_data() -> geopandas.GeoDataFrame:
 
 
 @pytest.fixture
-def toy_dem_data() -> (
-    tuple[geopandas.GeoDataFrame, geopandas.GeoDataFrame, geopandas.GeoDataFrame]
-):
+def toy_dem_data() -> tuple[
+    geopandas.GeoDataFrame, geopandas.GeoDataFrame, geopandas.GeoDataFrame
+]:
     """Toy demand data used in ``locate`` error & warning tests."""
 
     point = shapely.Point(10, 10)

spopt/tests/test_c_p_median.py

@@ -145,8 +145,7 @@ def test_infeasibility_predefined_facilities_fulfillment_error(self):
         with pytest.raises(
             SpecificationError,
             match=(
-                "Problem is infeasible. "
-                "The predefined facilities can't be fulfilled, "
+                "Problem is infeasible. The predefined facilities can't be fulfilled, "
             ),
         ):
             PMedian.from_cost_matrix(