python: Don't import pygame if not needed

JohnAZoidberg · JohnAZoidberg · commit ca30109f9ec6 · 2024-11-25T14:00:08.000+08:00
- Make pygame an optional dependency - Avoid the pygame hello print on startup ``` pygame 2.6.1 (SDL 2.28.4, Python 3.13.0) Hello from the pygame community. https://www.pygame.org/contribute.html ``` Signed-off-by: Daniel Schaefer <dhs@frame.work>
diff --git a/python/inputmodule/cli.py b/python/inputmodule/cli.py
@@ -62,10 +62,6 @@
     RGB_COLORS,
 )
 
-# Optional dependencies:
-# from PIL import Image
-# import PySimpleGUI as sg
-
 
 def main_cli():
     parser = argparse.ArgumentParser()
diff --git a/python/inputmodule/gui/__init__.py b/python/inputmodule/gui/__init__.py
@@ -18,7 +18,6 @@
     Game,
     GameControlVal
 )
-from inputmodule.gui.pygames import snake, ledris
 from inputmodule.gui.ledmatrix import countdown, random_eq, clock
 from inputmodule.gui.gui_threading import stop_thread, is_dev_disconnected
 from inputmodule.inputmodule.ledmatrix import (
@@ -233,12 +232,14 @@ def run_gui(devices):
     root.mainloop()
 
 def perform_action(devices, action):
-    action_map = {
-        "game_snake": snake.main_devices,
-        "game_ledris": ledris.main_devices,
-    }
-    if action in action_map:
-        threading.Thread(target=action_map[action], args=(devices,), daemon=True).start(),
+    if action.startswith("game_"):
+        from inputmodule.gui.pygames import snake, ledris
+        action_map = {
+            "game_snake": snake.main_devices,
+            "game_ledris": ledris.main_devices,
+        }
+        if action in action_map:
+            threading.Thread(target=action_map[action], args=(devices,), daemon=True).start(),
 
     if action == "bootloader":
         disable_devices(devices)

Original file line number	Diff line number	Diff line change
`@@ -62,10 +62,6 @@`
`62`	`62`	`RGB_COLORS,`
`63`	`63`	`)`
`64`	`64`
`65`		`-# Optional dependencies:`
`66`		`-# from PIL import Image`
`67`		`-# import PySimpleGUI as sg`
`68`		`-`
`69`	`65`
`70`	`66`	`def main_cli():`
`71`	`67`	`parser = argparse.ArgumentParser()`