mirror of
https://github.com/ChuckBuilds/LEDMatrix.git
synced 2026-10-04 06:15:09 +00:00
ChuckandClaude Opus 5.5
0d179fdf12
The frame loops and the dwell sleep run PluginManager.run_scheduled_updates() at most every 0.25 s instead of after every frame (the top of each loop pass still ticks unthrottled), and SportsScrollDisplay and the sync follower ask ScrollHelper.has_strip() instead of building cached_image just to see whether a strip exists. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
120 lines
4.5 KiB
Python
120 lines
4.5 KiB
Python
"""Follower adoption of the leader's scroll image (src/display_controller.py).
|
|
|
|
The leader's image arrives on the sync TCP thread. That callback used to set
|
|
scroll_helper.cached_image, cached_array and total_scroll_width one after
|
|
another while the render thread sliced frames out of them, so a frame could
|
|
pair the new array with the old width. The callback now only queues the
|
|
image; the render thread swaps all three in at the start of a follower frame.
|
|
"""
|
|
|
|
import os
|
|
from collections import deque
|
|
from types import SimpleNamespace
|
|
from unittest.mock import MagicMock, patch
|
|
|
|
os.environ.setdefault("EMULATOR", "true")
|
|
|
|
from PIL import Image # noqa: E402
|
|
|
|
from src.display_controller import DisplayController # noqa: E402
|
|
|
|
|
|
def _wired_controller():
|
|
dc = object.__new__(DisplayController)
|
|
dc.config = {"display": {"vegas_scroll": {"enabled": True}}, "sync": {}}
|
|
dc.display_manager = MagicMock()
|
|
dc.plugin_manager = MagicMock()
|
|
dc.sync_manager = MagicMock()
|
|
dc._check_live_priority = MagicMock()
|
|
dc._check_vegas_interrupt = MagicMock(return_value=False)
|
|
dc._follower_pending_new_image = True
|
|
dc._follower_incoming_image = deque(maxlen=1)
|
|
|
|
old = Image.new("RGB", (100, 16))
|
|
helper = SimpleNamespace(cached_image=old, cached_array="old-array",
|
|
total_scroll_width=100)
|
|
coordinator = MagicMock()
|
|
coordinator.render_pipeline = SimpleNamespace(scroll_helper=helper)
|
|
with patch('src.vegas_mode.VegasModeCoordinator',
|
|
MagicMock(return_value=coordinator)):
|
|
dc._initialize_vegas_mode()
|
|
on_image = dc.sync_manager.set_on_scroll_image.call_args[0][0]
|
|
return dc, coordinator.render_pipeline, helper, old, on_image
|
|
|
|
|
|
def test_tcp_callback_does_not_touch_the_render_state():
|
|
dc, _rp, helper, old, on_image = _wired_controller()
|
|
on_image(Image.new("RGB", (300, 16)))
|
|
# Nothing the render thread reads has changed yet.
|
|
assert helper.cached_image is old
|
|
assert helper.cached_array == "old-array"
|
|
assert helper.total_scroll_width == 100
|
|
assert dc._follower_pending_new_image is True
|
|
|
|
|
|
def test_render_thread_adopts_all_three_together():
|
|
dc, rp, helper, _old, on_image = _wired_controller()
|
|
new = Image.new("RGB", (300, 16), (255, 0, 0))
|
|
on_image(new)
|
|
dc._adopt_follower_scroll_image(rp)
|
|
assert helper.cached_image is new
|
|
assert helper.cached_array.shape == (16, 300, 3)
|
|
assert helper.cached_array[0, 0].tolist() == [255, 0, 0]
|
|
assert helper.total_scroll_width == 300
|
|
assert dc._follower_pending_new_image is False
|
|
# Consumed: a second frame changes nothing.
|
|
helper.total_scroll_width = 7
|
|
dc._adopt_follower_scroll_image(rp)
|
|
assert helper.total_scroll_width == 7
|
|
|
|
|
|
def test_only_the_latest_image_is_adopted():
|
|
dc, rp, helper, _old, on_image = _wired_controller()
|
|
on_image(Image.new("RGB", (200, 16)))
|
|
latest = Image.new("RGB", (400, 16))
|
|
on_image(latest)
|
|
dc._adopt_follower_scroll_image(rp)
|
|
assert helper.cached_image is latest
|
|
assert helper.total_scroll_width == 400
|
|
|
|
|
|
def test_a_follower_frame_does_not_build_a_deferred_strip_image():
|
|
"""Every follower frame asks whether the strip is there. A strip the
|
|
follower's own rebuild deferred (append_content) must be answered from
|
|
the helper's bookkeeping, not by building and keeping its PIL image."""
|
|
import numpy as np
|
|
|
|
from src.common.scroll_helper import ScrollHelper
|
|
|
|
width, height = 64, 16
|
|
helper = ScrollHelper(width, height)
|
|
helper.append_content([Image.new("RGB", (300, height), (0, 200, 0))])
|
|
helper.append_content([Image.new("RGB", (300, height), (0, 0, 200))])
|
|
assert helper.__dict__.get("_cached_image") is None
|
|
assert helper.has_strip()
|
|
|
|
dc = object.__new__(DisplayController)
|
|
dc.config = {"sync": {}}
|
|
dc.vegas_coordinator = SimpleNamespace(
|
|
render_pipeline=SimpleNamespace(scroll_helper=helper))
|
|
dc.display_manager = MagicMock(width=width)
|
|
dc.sync_manager = MagicMock()
|
|
dc.sync_manager.get_latest_scroll_x.return_value = 200
|
|
dc._follower_incoming_image = deque(maxlen=1)
|
|
dc._follower_dr_last_t = None
|
|
dc._follower_local_x = 200.0
|
|
dc._follower_pending_new_image = False
|
|
dc._follower_last_frame = None
|
|
dc._follower_deadline = None
|
|
dc._scroll_speed = 0
|
|
|
|
for _ in range(3):
|
|
dc._run_follower_frame()
|
|
|
|
# The frame was cut from the array...
|
|
frame = np.asarray(dc._follower_last_frame)
|
|
assert frame.shape == (height, width, 3)
|
|
assert frame.any()
|
|
# ...and the strip is still held once.
|
|
assert helper.__dict__.get("_cached_image") is None
|