Migrate from black to ruff

Author: VirxEC

README.md

@@ -30,10 +30,11 @@ The following is how to setup a development environment for this project, NOT ho
   - `pip uninstall rlbot_flatbuffers`
   - `pip install --editable <path/to/rlbot_flatbuffers>`
 
-This project is formatted using [Black](https://github.com/psf/black).
+This project is formatted using [Ruff](https://docs.astral.sh/ruff/formatter/).
 
-- Install: `pip install black`.
-- Use: `black .`
+- Install: `pip install ruff`.
+- Sort imports: `ruff check --select I --fix`
+- Format code: `ruff format`
 
 ## Testing
 

pyproject.toml

@@ -23,3 +23,12 @@ Repository = "https://github.com/RLBot/python-interface"
 
 [tool.setuptools.dynamic]
 version = {attr = "rlbot.version.__version__"}
+
+[dependency-groups]
+dev = [
+    "ruff>=0.12.5",
+    "toml>=0.10.2",
+]
+
+[tool.ruff.format]
+docstring-code-format = true

rlbot/__init__.py

@@ -1 +1 @@
-from .version import __version__
+from .version import __version__ as __version__

rlbot/interface.py

@@ -323,7 +323,7 @@ def disconnect(self):
             self.logger.critical("RLBot is not responding to our disconnect request!?")
             self._running = False
 
-        assert (
-            not self._running
-        ), "Disconnect request or timeout should have set self._running to False"
+        assert not self._running, (
+            "Disconnect request or timeout should have set self._running to False"
+        )
         self.is_connected = False

rlbot/utils/maps.py

@@ -83,7 +83,7 @@
     "FuturaGarden": "UF_Day_P",
     "DFHStadium_Anniversary": "stadium_10a_p",
     "Holyfield": "Labs_Holyfield_Space_P",
-    "DriftWoods_Night": "woods_night_p"
+    "DriftWoods_Night": "woods_night_p",
 }
 
 STANDARD_MAPS = [
@@ -138,5 +138,5 @@
     "Neotokyo_Arcade",
     "FuturaGarden",
     "DFHStadium_Anniversary",
-    "DriftWoods_Night"
+    "DriftWoods_Night",
 ]

tests/fashion/bot.py

@@ -50,7 +50,7 @@ def get_output(self, packet: flat.GamePacket) -> flat.ControllerState:
                 ),
             )
 
-            self.logger.info(f"State setting new loadout")
+            self.logger.info("State setting new loadout")
             self.set_loadout(loadout)
             self.last_tick = packet.match_info.frame_num
 

tests/nexto/bot.py

@@ -369,7 +369,6 @@ def toxicity(self, packet: GamePacket):
                 return
 
             for p in human_opps:
-
                 d = math.sqrt(
                     (p.physics.location.x - bad_goal[0]) ** 2
                     + (p.physics.location.y - bad_goal[1]) ** 2

tests/nexto/nexto_obs.py

@@ -286,8 +286,7 @@ def batched_build_obs(self, encoded_states: np.ndarray):
         for i in range(n_players):
             encoded_player = encoded_states[
                 :,
-                players_start_index
-                + i * player_length : players_start_index
+                players_start_index + i * player_length : players_start_index
                 + (i + 1) * player_length,
             ]
 

tests/run_forever.py

@@ -13,7 +13,6 @@
 
 if __name__ == "__main__":
     match_manager = MatchManager(RLBOT_SERVER_FOLDER)
-    match_manager.ensure_server_started()
 
     current_map = -1
 

tests/runner.py

@@ -16,7 +16,6 @@
     match_manager = MatchManager(RLBOT_SERVER_FOLDER)
 
     try:
-        match_manager.ensure_server_started()
         match_manager.start_match(MATCH_CONFIG_PATH)
 
         logger.info("Waiting before shutdown...")
@@ -25,7 +24,7 @@
     except KeyboardInterrupt:
         logger.warning("Shutting down early due to interrupt")
     except Exception:
-        logger.critical(f"Shutting down early due to the following error:")
+        logger.critical("Shutting down early due to the following error:")
         print_exc()
 
     match_manager.shut_down()