Merge pull request #88 from funkelab/update-precommit

Use ruff-format, add Caroline to authors

.pre-commit-config.yaml

@@ -5,18 +5,14 @@ ci:
 
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.3.4
+    rev: v0.4.2
     hooks:
       - id: ruff
         args: [--fix, --unsafe-fixes]
-
-  - repo: https://github.com/psf/black
-    rev: 24.3.0
-    hooks:
-      - id: black
+      - id: ruff-format
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.9.0
+    rev: v1.10.0
     hooks:
       - id: mypy
         files: "^motile/"

motile/costs/edge_distance.py

@@ -55,7 +55,6 @@ def apply(self, solver: Solver) -> None:
             solver.add_variable_cost(index, 1.0, self.constant)
 
     def __get_node_position(self, graph: nx.DiGraph, node: int) -> np.ndarray:
-
         if isinstance(self.position_attribute, tuple):
             return np.array([graph.nodes[node][p] for p in self.position_attribute])
         else:

pyproject.toml

@@ -13,6 +13,7 @@ license = { text = "MIT" }
 authors = [
     { name = 'Jan Funke', email = '[email protected]' },
     { name = 'Talley Lambert', email = '[email protected]' },
+    { name = 'Caroline Malin-Mayor', email = '[email protected]' },
     { name = 'Benjamin Gallusser', email = '[email protected]' },
     { name = 'Florian Jug', email = '[email protected]' },
 ]
@@ -44,6 +45,7 @@ path = "motile/__init__.py"
 [tool.ruff]
 target-version = "py38"
 src = ["motile"]
+
 [tool.ruff.lint]
 select = [
     "F",   # pyflakes
@@ -58,13 +60,12 @@ ignore = [
     "D104", # Missing docstring in public package
     "D105", # Missing docstring in magic method
     "D107", # Missing docstring in `__init__`
-
     "D102", # Missing docstring in public method
-    # "D205", # 1 blank line required between summary line and description
-
 ]
+
 [tool.ruff.lint.pydocstyle]
 convention = "google"
+
 [tool.ruff.lint.per-file-ignores]
 "tests/*" = ["D"]
 

tests/test_constraints.py

@@ -67,7 +67,6 @@ def test_max_parents(solver: motile.Solver) -> None:
 
 
 def test_exlusive_nodes(solver: motile.Solver) -> None:
-
     exclusive_sets = [
         [0, 1],
         [2, 3],