ruff

Author: jquast

gambaterm/controller_input.py

@@ -51,13 +51,11 @@ def pygame_button_pressed_context(
     try:
         import pygame
     except ImportError:
-        raise SystemExit(
-            """\
+        raise SystemExit("""\
 The pygame library is not available, which is required to get controller support.
 Please use the following command to install gambaterm with controller support:
   pip3 install gambaterm[controller-support]
-"""
-        )
+""")
 
     pygame.init()
     pygame.joystick.init()

gambaterm/graphics_scaler.py

@@ -24,12 +24,8 @@
 
 BASELINE_ID = 1
 DELTA_ID = 2
-SIXEL_REBASELINE_THRESHOLD = float(
-    os.environ.get("GAMBATERM_SIXEL_REBASELINE", "0.35")
-)
-KITTY_REBASELINE_THRESHOLD = float(
-    os.environ.get("GAMBATERM_KITTY_REBASELINE", "0.35")
-)
+SIXEL_REBASELINE_THRESHOLD = float(os.environ.get("GAMBATERM_SIXEL_REBASELINE", "0.35"))
+KITTY_REBASELINE_THRESHOLD = float(os.environ.get("GAMBATERM_KITTY_REBASELINE", "0.35"))
 SCALE_MAX = int(os.environ.get("GAMBATERM_SCALE_MAX", "12"))
 
 
@@ -239,9 +235,7 @@ def __init__(
                     "x1,y1,rect_w,rect_h,"
                     "refx_kitty,refy_kitty,scale\n"
                 )
-            atexit.register(
-                self.profile.dump, f"{profile_dir}/gambaterm-profile.txt"
-            )
+            atexit.register(self.profile.dump, f"{profile_dir}/gambaterm-profile.txt")
         else:
             self.stats_fh = open(os.devnull, "w")
 
@@ -531,7 +525,10 @@ def encode_kitty_frame(self, video, encode_fn):
         rect_h = y2 - y1
         rect_area = rect_w * rect_h
 
-        if changed_pct > KITTY_REBASELINE_THRESHOLD or rect_area > total_pixels * KITTY_REBASELINE_THRESHOLD:
+        if (
+            changed_pct > KITTY_REBASELINE_THRESHOLD
+            or rect_area > total_pixels * KITTY_REBASELINE_THRESHOLD
+        ):
             self.profile.keyframes += 1
             self.baseline = video.copy()
             rgba = to_rgba_u8(video)

gambaterm/main.py

@@ -14,7 +14,12 @@
 from .console import GameboyColor, Console
 from .audio import audio_player
 from .colors import detect_local_color_mode, ColorMode
-from .remote_terminal import GraphicsProtocol, FORCE_SIXEL_BLITLESS, BAD_TEXT, does_sixel
+from .remote_terminal import (
+    GraphicsProtocol,
+    FORCE_SIXEL_BLITLESS,
+    BAD_TEXT,
+    does_sixel,
+)
 from .graphics_scaler import parse_autoscale
 from .keyboard_input import is_kitty_keyboard_protocol_supported
 from .input_getter import BaseInputGetter

gambaterm/run.py

@@ -168,7 +168,8 @@ def run(
                 cycle = [
                     p
                     for p in graphics_cycle
-                    if p is not GraphicsProtocol.TEXT or not terminal_name.startswith(BAD_TEXT)
+                    if p is not GraphicsProtocol.TEXT
+                    or not terminal_name.startswith(BAD_TEXT)
                 ]
                 if cycle:
                     idx = cycle.index(graphics_protocol)
@@ -180,7 +181,10 @@ def run(
                 average_over = int(round(fps))  # frames
                 audio_out.update_speed(console, speed)
             elif key.key_name in ("FOCUS_IN", "FOCUS_OUT"):
-                if graphics_protocol is GraphicsProtocol.SIXEL and terminal_name.startswith("mlterm"):
+                if (
+                    graphics_protocol is GraphicsProtocol.SIXEL
+                    and terminal_name.startswith("mlterm")
+                ):
                     if scaler is not None:
                         scaler.sixel_baseline = None
             elif key.key_name in ("KEY_GRAVE_ACCENT", "KEY_TILDE") or key in ("`", "~"):
@@ -312,15 +316,11 @@ def run(
                     force_clear = False
                     frame_data += maybe_clear_sequence
                     if graphics_protocol is GraphicsProtocol.SIXEL:
-                        frame_data += scaler.blit_sixel(
-                            video, last_frame
-                        )
+                        frame_data += scaler.blit_sixel(video, last_frame)
                     elif graphics_protocol is GraphicsProtocol.BLITLESS_SIXEL:
                         frame_data += scaler.blit_sixel_blitless(video, width, height)
                     else:
-                        frame_data += scaler.blit_kitty(
-                            video, last_frame
-                        )
+                        frame_data += scaler.blit_kitty(video, last_frame)
                         if terminal_name.startswith(FORCE_KITTY_BLITLESS):
                             scaler.baseline = None
                     if sync_end:

gambaterm/ssh.py

@@ -199,10 +199,7 @@ def ssh_terminal_handler(
     input_source = detect_input_source(app_config, keyboard_support_detection)
     console_input_context: ContextManager[BaseInputGetter]
     if input_source is None:
-        message = (
-            MESSAGE_SUGGESTING_KITTY_SUPPORT
-            + "\n\n"
-            + """\
+        message = MESSAGE_SUGGESTING_KITTY_SUPPORT + "\n\n" + """\
 Alternatively, X11 forwarding can be used in order to give the gambaterm-ssh
 server access to your keyboard, eg. `ssh -Y -p 8022 localhost`.
 ===============================[ WARNING ]=====================================
@@ -211,7 +208,6 @@ def ssh_terminal_handler(
 sandbox. More information here: https://security.stackexchange.com/a/7496
 ===============================[ WARNING ]=====================================
 """
-        )
         terminal.stream.write(message)
         terminal.stream.flush()
         session_logger.warning(
@@ -338,10 +334,10 @@ def __init__(
         users_directory: Path,
         executor: ThreadPoolExecutor,
         active_connections: dict[GambatermSSHServer, SSHServerConnection],
-        frontend: Callable[
-            [RemoteTerminal, AppConfig, KeyboardSupportDetection], AppConfig
-        ]
-        | None = None,
+        frontend: (
+            Callable[[RemoteTerminal, AppConfig, KeyboardSupportDetection], AppConfig]
+            | None
+        ) = None,
     ):
         self._gambaterm_console_cls = console_cls
         self._gambaterm_namespace = namespace

tests/_frontend_ssh_server.py

@@ -9,7 +9,6 @@
 from gambaterm.main import AppConfig
 from gambaterm.remote_terminal import RemoteTerminal, KeyboardSupportDetection
 
-
 rom = Path(sys.argv[1])
 
 

tests/_frontend_telnet_server.py

@@ -9,7 +9,6 @@
 from gambaterm.main import AppConfig
 from gambaterm.remote_terminal import RemoteTerminal, KeyboardSupportDetection
 
-
 rom = Path(sys.argv[1])
 
 

tests/test_gambaterm.py

@@ -66,8 +66,9 @@ def test_gambaterm_ssh(
     ssh_config: Path, gambaterm_config: Path, color_arg: str
 ) -> None:
     assert TEST_ROM.exists()
-    command = f"gambaterm-ssh {TEST_ROM} --break-after 10 --input-file /dev/null --graphics text" + (
-        f" {color_arg}" if color_arg else ""
+    command = (
+        f"gambaterm-ssh {TEST_ROM} --break-after 10 --input-file /dev/null --graphics text"
+        + (f" {color_arg}" if color_arg else "")
     )
     env = os.environ.copy()
     env["PYTHONUNBUFFERED"] = "1"
@@ -125,7 +126,8 @@ def test_gambaterm_telnet(color_arg: str) -> None:
     assert TEST_ROM.exists()
     command = (
         f"{sys.executable} -m gambaterm.telnet {TEST_ROM} --break-after 10"
-        f" --input-file /dev/null --graphics text" + (f" {color_arg}" if color_arg else "")
+        f" --input-file /dev/null --graphics text"
+        + (f" {color_arg}" if color_arg else "")
     )
     env = os.environ.copy()
     env["PYTHONUNBUFFERED"] = "1"

tests/test_graphics_renderer.py

@@ -202,6 +202,7 @@ def test_sixel_rebaseline_on_large_change():
     assert "\033P0;1;0q" in text
     assert "\033\\" in text
 
+
 class TestAutoScale:
     @staticmethod
     def test_fps_below_threshold_reduces():
@@ -311,12 +312,15 @@ def test_disabled(self, value):
 
         assert parse_autoscale(value).enabled is False
 
-    @pytest.mark.parametrize("value,seconds,fps,mbits", [
-        ("30fps", -1, 30.0, 0.0),
-        ("60s,30fps,10mb", 60, 30.0, 80.0),
-        ("1500kb", -1, 40.0, 12.0),
-        ("always,25fps", -1, 25.0, 0.0),
-    ])
+    @pytest.mark.parametrize(
+        "value,seconds,fps,mbits",
+        [
+            ("30fps", -1, 30.0, 0.0),
+            ("60s,30fps,10mb", 60, 30.0, 80.0),
+            ("1500kb", -1, 40.0, 12.0),
+            ("always,25fps", -1, 25.0, 0.0),
+        ],
+    )
     def test_parse(self, value, seconds, fps, mbits):
         from gambaterm.graphics_scaler import parse_autoscale
 
@@ -326,11 +330,14 @@ def test_parse(self, value, seconds, fps, mbits):
         assert cfg.fps == fps
         assert cfg.bandwidth_mbits == mbits
 
-    @pytest.mark.parametrize("value", [
-        "always,90s,30fps",
-        "disabled,90s,10mb",
-        "off,30fps",
-    ])
+    @pytest.mark.parametrize(
+        "value",
+        [
+            "always,90s,30fps",
+            "disabled,90s,10mb",
+            "off,30fps",
+        ],
+    )
     def test_mutually_exclusive_errors(self, value):
         from gambaterm.graphics_scaler import parse_autoscale