Update Ruff to 0.3.4; apply ruff format . (projectmesa#2088)

* Update Ruff to 0.3.4; apply ruff format .

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

.pre-commit-config.yaml

@@ -4,7 +4,7 @@ ci:
 repos:
 - repo: https://github.com/astral-sh/ruff-pre-commit
   # Ruff version.
-  rev: v0.2.0
+  rev: v0.3.4
   hooks:
     # Run the linter.
     - id: ruff

benchmarks/WolfSheep/wolf_sheep.py

@@ -38,8 +38,7 @@ def spawn_offspring(self):
         offspring.move_to(self.cell)
         self.model.schedule.add(offspring)
 
-    def feed(self):
-        ...
+    def feed(self): ...
 
     def die(self):
         self.cell.remove_agent(self)

mesa/__init__.py

@@ -3,6 +3,7 @@
 
 Core Objects: Model, and Agent.
 """
+
 import datetime
 
 import mesa.space as space

mesa/agent.py

@@ -3,6 +3,7 @@
 
 Core Objects: Agent
 """
+
 # Mypy; for the `|` operator purpose
 # Remove this __future__ import once the oldest supported Python is 3.10
 from __future__ import annotations

mesa/datacollection.py

@@ -32,6 +32,7 @@
     * The model has an agent list called agents
     * For collecting agent-level variables, agents must have a unique_id
 """
+
 import contextlib
 import itertools
 import types

mesa/experimental/cell_space/discrete_space.py

@@ -43,8 +43,7 @@ def __init__(
     def cutoff_empties(self):
         return 7.953 * len(self._cells) ** 0.384
 
-    def _connect_single_cell(self, cell: T):
-        ...
+    def _connect_single_cell(self, cell: T): ...
 
     @cached_property
     def all_cells(self):

mesa/experimental/cell_space/grid.py

@@ -51,11 +51,9 @@ def _connect_cells(self) -> None:
         else:
             self._connect_cells_nd()
 
-    def _connect_cells_2d(self) -> None:
-        ...
+    def _connect_cells_2d(self) -> None: ...
 
-    def _connect_cells_nd(self) -> None:
-        ...
+    def _connect_cells_nd(self) -> None: ...
 
     def _validate_parameters(self):
         if not all(isinstance(dim, int) and dim > 0 for dim in self.dimensions):

mesa/model.py

@@ -3,6 +3,7 @@
 
 Core Objects: Model
 """
+
 # Mypy; for the `|` operator purpose
 # Remove this __future__ import once the oldest supported Python is 3.10
 from __future__ import annotations
@@ -55,7 +56,7 @@ def __new__(cls, *args: Any, **kwargs: Any) -> Any:
         if obj._seed is None:
             # We explicitly specify the seed here so that we know its value in
             # advance.
-            obj._seed = random.random()  # noqa: S311
+            obj._seed = random.random()
         obj.random = random.Random(obj._seed)
         # TODO: Remove these 2 lines just before Mesa 3.0
         obj._steps = 0

mesa/space.py

@@ -150,14 +150,12 @@ def build_empties(self) -> None:
         self._empties_built = True
 
     @overload
-    def __getitem__(self, index: int | Sequence[Coordinate]) -> list[GridContent]:
-        ...
+    def __getitem__(self, index: int | Sequence[Coordinate]) -> list[GridContent]: ...
 
     @overload
     def __getitem__(
         self, index: tuple[int | slice, int | slice]
-    ) -> GridContent | list[GridContent]:
-        ...
+    ) -> GridContent | list[GridContent]: ...
 
     def __getitem__(self, index):
         """Access contents from the grid."""
@@ -420,11 +418,9 @@ def get_cell_list_contents(self, cell_list: Iterable[Coordinate]) -> list[Agent]
         """
         return list(self.iter_cell_list_contents(cell_list))
 
-    def place_agent(self, agent: Agent, pos: Coordinate) -> None:
-        ...
+    def place_agent(self, agent: Agent, pos: Coordinate) -> None: ...
 
-    def remove_agent(self, agent: Agent) -> None:
-        ...
+    def remove_agent(self, agent: Agent) -> None: ...
 
     def move_agent(self, agent: Agent, pos: Coordinate) -> None:
         """Move an agent from its current position to a new position.

pyproject.toml

@@ -132,4 +132,5 @@ extend-ignore = [
   "S310",   # Audit URL open for permitted schemes. Allowing use of `file:` or custom schemes is often unexpected.
   "S603",   # `subprocess` call: check for execution of untrusted input
   "ISC001", # ruff format asks to disable this feature
+  "S311",   # Standard pseudo-random generators are not suitable for cryptographic purposes
 ]

tests/test_datacollector.py

@@ -1,6 +1,7 @@
 """
 Test the DataCollector
 """
+
 import unittest
 
 from mesa import Agent, Model

tests/test_grid.py

@@ -1,6 +1,7 @@
 """
 Test the Grid objects.
 """
+
 import random
 import unittest
 from unittest.mock import Mock, patch