mirror of
https://github.com/ChuckBuilds/LEDMatrix.git
synced 2026-10-04 06:15:09 +00:00
* perf(timing): say which render-thread work a late frame followed The soak already says how often a moving frame reached the panel late, but not what the render thread was doing just before it. Vegas does two kinds of work there between frames -- building its strip (compose, extend) and, with live elements, patching changed pixels into it -- and deciding whether either is affordable needs their own numbers. - FrameTimingRecorder.note_op(kind, nbytes) tags the next presented frame. Totals gain op_frames, late_op_frames, op_freezes and op_bytes per kind; aggregate() still takes frames without ops. The file schema is unchanged. - Vegas tags compose and every strip extension (with the bytes it copied). - frame_soak prints an "after work" table: frames, late %, freezes and MB moved per kind, only when something tagged its work. - render_bench gains --strip-screens (Vegas-sized strips), --patch-bytes / --patch-every / --patch-where (in-place column writes, as a live element update does) and --extend-every-screens / --extend-width (append + trim on a fixed cadence that holds the strip's width). No runtime behaviour changes: this is the measurement gate for live Vegas elements. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(changelog): note the frame-op attribution and bench modes Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * perf(scroll): build the strip's PIL image only when something reads it Every Vegas strip extension rebuilt ScrollHelper.cached_image from cached_array in full, twice (append, then trim), on the render thread: Image.fromarray is 1.7ms for an 8,000px strip and 3.8ms for 20,000px on a Pi 4 (measured on ledpi), about two thirds of an extension's render-thread cost. Nothing on the frame path reads the image's pixels; every frame is cut from the array. cached_image is now a property. append_content and drop_scrolled_prefix defer it; the first read builds it from the array it started with and keeps it only if the strip has not changed meanwhile, so a sync push racing an extension cannot leave a stale image cached. Assigning cached_image stores exactly what was assigned, as before. has_strip() says whether there is a strip without building its image; the helper's frame path, Vegas and the adapter's scroll-cache invalidation use it. The strip is also no longer held in memory twice. In Vegas the image is now built only by a multi-display sync push. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * feat(vegas): live elements -- a plugin API for content that changes while it scrolls Vegas bakes each plugin's pictures into one strip, so a card already on its way across the panel keeps what it showed when it was drawn. This adds the API and bookkeeping for content that can be updated in place; the worker that redraws and swaps it follows separately. No shipped plugin implements the hook yet, so nothing changes for users. Plugin API (core 3.8.0), all no-ops by default: - BasePlugin.get_vegas_elements() -> [VegasElement(key, image, version, live, refresh_hz)]: named, fixed-width pieces of Vegas content. - BasePlugin.redraw_vegas_element(key, width, height, at): a lock-free redraw for content that changes with time. - BasePlugin.notify_vegas_data_changed(): data that lands outside update(). - src/plugin_system/vegas_elements.py (VegasElement, re-exported from base_plugin). Core: - PluginAdapter asks a plugin that implements the hook for elements on the background fetch only (under its lock, on its own canvas); every other path keeps get_vegas_content(). Live elements are pinned (padded with content_padding, never trimmed), tagged with their key, digest and data epoch in Image.info so the existing cache and group plumbing carry them unchanged, and untagged if a width budget crops them. - RenderPipeline records where each live element lands (ElementRecord), in absolute strip columns a trim does not move; the block-start arithmetic is shared with the STATIC markers. - PluginManager update listeners (add/remove_update_listener, notify_data_changed): told the moment update() completes, not at the next ~4s Vegas poll. The coordinator uses one to move each plugin's data epoch on. - vegas_scroll.live_refresh (kill switch), live_max_hz, live_min_interval, live_lead_screens; per-plugin core-owned vegas_live. Live elements are off under multi-display sync, in swap mode and with offscreen_prefetch off. - scripts/check_plugin.py checks the element contract (src/plugin_system/testing/vegas.py); test/fixtures/plugins/vegas-live-stub is a working example. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * feat(vegas): live elements update in place while they scroll One background worker (src/vegas_mode/live_worker.py) redraws a plugin's live elements when its data epoch moves on (update listener) or on their refresh_hz, nearest the screen first, and hands changed pixels lock-free to the render thread, which copies them into the strip between frames (RenderPipeline.apply_live_patches, ScrollHelper.patch_columns): at most four patches or two screens of bytes a frame, no drawing or locks there. The worker takes over group prefetch once a live element is placed, runs inside the render gate, and is supervised. Update tick 1s while live elements exist. Web UI switch for live_refresh. OFFSCREEN_RENDERING.md describes what was built and why SegmentStrip was not needed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * feat(sports): live Vegas cards for the scoreboards (shared layer) One live element per game, drawn only when what the card shows changes, so a score changes on a card already crossing the panel. The shared part, so each scoreboard adopts it in a few lines: - src/common/sports_vegas.py: game_key, game_fingerprint (the whole game dict, frozen: no drawn field can be missed), dedupe_games, VegasCardCache, StickyOdds (odds a live poll left out stay drawn), finished_games / with_finished_games (a game that just went final keeps its card, after its league's live games; one a heuristic only judged over keeps its live state, so a tied end of regulation never shows FINAL early). - SportsScrollDisplay.make_vegas_renderer() is the override point; build_vegas_elements() and SportsScrollDisplayManager .get_vegas_elements_for() do the rest. A card's version includes its teams' ranks, which the renderer draws from the rankings cache. - SportsLiveSharedMixin._record_finished_game() / finished_games_snapshot(): held for FINISHED_GAME_TTL after it leaves the live list. A sport that does not implement make_vegas_renderer keeps its ordinary Vegas content, so no scoreboard changes until it opts in. scripts/render_plugin.py --vegas renders a plugin's Vegas block as the ticker lays it out, and --timeline stacks it at successive moments as the ticker would update it in place; the join is now render_pipeline.join_plugin_rows(). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * feat(vegas): keep live games in the ticker by default display.vegas_scroll.live_in_ticker now defaults to true: through a live game the marquee keeps running and the live scoreboard takes extra turns in it -- its cards updating in place while they scroll -- instead of the ticker giving way to the full-screen scoreboard. The new default would reach nobody on its own: every existing config holds an explicit false copied from the template (there was no control for it), and the template merge only adds missing keys. ConfigManager therefore turns a stored false on once, with a backup, and records live_in_ticker_migrated so a false chosen afterwards stays. The marker is never in the template. A "Keep live games in the ticker" checkbox under Vegas mode sets it. Tests that pin the full-screen takeover now say live_in_ticker=false. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * refactor(sports): a default _determine_game_type on SportsScrollDisplay render_vegas_card looked the method up with getattr and a None default, which static analysis (Codacy) reports as calling something that may not be callable. The base class now has the default -- the card type from the game's state -- and the plugins that define their own override it as before. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix: review follow-ups on the shared live-card layer - The reused Vegas renderer always gets the current rankings, empty included, so ranks cleared since are not kept drawn. - render_plugin.py: --timeline refuses --no-live (a timeline shows live elements changing), --timeline/--no-live need --vegas, and the Vegas paths create the output's directory like the display path does. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
301 lines
13 KiB
Python
301 lines
13 KiB
Python
"""Tests that live content can take extra turns inside the Vegas ticker.
|
|
|
|
Vegas was a strict round robin -- every plugin exactly once per cycle -- and
|
|
live content did not appear in it at all, because the display controller
|
|
refused to run the ticker while anything was live. With a dozen plugins
|
|
enabled that left a live score either absent or minutes stale.
|
|
|
|
Two things change. `live_in_ticker` keeps the marquee running instead of
|
|
yielding to a full-screen takeover (on by default since 3.8.0), and the
|
|
rotation is expanded by Smooth Weighted Round-Robin so a weighted plugin gets
|
|
several slots per cycle, spaced through it rather than clumped.
|
|
|
|
Weights are per plugin, not per game: a scoreboard showing four live games
|
|
still occupies one slot at a time and rotates its own games within it.
|
|
"""
|
|
|
|
from unittest.mock import Mock
|
|
|
|
import pytest
|
|
|
|
from src.vegas_mode.config import VegasModeConfig
|
|
from src.vegas_mode.stream_manager import StreamManager
|
|
|
|
|
|
class FakePlugin:
|
|
"""A plugin that can fail in each place independently.
|
|
|
|
hook_raises and live_raises are separate because they mean different
|
|
things: a broken weight calculation should still leave the core's own
|
|
live-content check usable, while a plugin that cannot answer whether it is
|
|
live at all has nothing left to fall back on.
|
|
"""
|
|
|
|
def __init__(self, live=False, declared=None, raises=False,
|
|
hook_raises=False, live_raises=False):
|
|
self._live = live
|
|
self._declared = declared
|
|
self._hook_raises = hook_raises or raises
|
|
self._live_raises = live_raises or raises
|
|
self.enabled = True
|
|
|
|
def has_live_priority(self):
|
|
if self._live_raises:
|
|
raise RuntimeError("cannot say whether I am live")
|
|
return self._live
|
|
|
|
def has_live_content(self):
|
|
return self._live
|
|
|
|
def get_vegas_priority_weight(self):
|
|
if self._hook_raises:
|
|
raise RuntimeError("weight calculation blew up")
|
|
return self._declared
|
|
|
|
|
|
def _manager(plugins, **cfg):
|
|
config = VegasModeConfig(live_in_ticker=cfg.pop('live_in_ticker', True), **cfg)
|
|
pm = Mock()
|
|
pm.plugins = plugins
|
|
sm = StreamManager.__new__(StreamManager)
|
|
sm.config = config
|
|
sm.plugin_manager = pm
|
|
return sm
|
|
|
|
|
|
def _counts(schedule):
|
|
return {p: schedule.count(p) for p in set(schedule)}
|
|
|
|
|
|
def _max_gap(schedule, plugin_id):
|
|
"""Largest gap between consecutive appearances, wrapping around."""
|
|
at = [i for i, p in enumerate(schedule) if p == plugin_id]
|
|
if len(at) < 2:
|
|
return len(schedule)
|
|
gaps = [b - a for a, b in zip(at, at[1:])]
|
|
gaps.append(len(schedule) - at[-1] + at[0])
|
|
return max(gaps)
|
|
|
|
|
|
class TestWeightsComeFromTheRightPlace:
|
|
def test_a_quiet_plugin_gets_one_slot(self):
|
|
sm = _manager({'clock': FakePlugin()})
|
|
assert sm._plugin_weight('clock') == 1
|
|
|
|
def test_live_content_earns_the_configured_weight(self):
|
|
sm = _manager({'mlb': FakePlugin(live=True)}, live_weight=4)
|
|
assert sm._plugin_weight('mlb') == 4
|
|
|
|
def test_a_plugin_may_answer_for_itself(self):
|
|
# The only route for favorite-team awareness: the core can see that a
|
|
# game is live, not whose.
|
|
sm = _manager({'mlb': FakePlugin(live=True, declared=7)}, live_weight=3)
|
|
assert sm._plugin_weight('mlb') == 7
|
|
|
|
def test_declaring_none_defers_to_the_core(self):
|
|
sm = _manager({'mlb': FakePlugin(live=True, declared=None)}, live_weight=3)
|
|
assert sm._plugin_weight('mlb') == 3
|
|
|
|
def test_a_declared_weight_is_clamped(self):
|
|
sm = _manager({'a': FakePlugin(declared=99), 'b': FakePlugin(declared=0)})
|
|
assert sm._plugin_weight('a') == 10
|
|
assert sm._plugin_weight('b') == 1
|
|
|
|
def test_a_plugin_that_raises_everywhere_weighs_one(self):
|
|
sm = _manager({'bad': FakePlugin(raises=True)})
|
|
assert sm._plugin_weight('bad') == 1
|
|
|
|
def test_a_broken_hook_still_earns_the_live_boost(self):
|
|
# The hook is only how a plugin asks for *more* than live_weight.
|
|
# Losing it should cost the favorite distinction, not the live boost:
|
|
# has_live_priority/has_live_content are separate and still work.
|
|
sm = _manager({'mlb': FakePlugin(live=True, hook_raises=True)},
|
|
live_weight=4)
|
|
assert sm._plugin_weight('mlb') == 4
|
|
|
|
def test_a_broken_hook_on_a_quiet_plugin_weighs_one(self):
|
|
sm = _manager({'clock': FakePlugin(live=False, hook_raises=True)},
|
|
live_weight=4)
|
|
assert sm._plugin_weight('clock') == 1
|
|
|
|
def test_a_plugin_that_cannot_say_whether_it_is_live_weighs_one(self):
|
|
# Nothing left to fall back on, so no boost.
|
|
sm = _manager({'mlb': FakePlugin(live=True, live_raises=True)},
|
|
live_weight=4)
|
|
assert sm._plugin_weight('mlb') == 1
|
|
|
|
def test_an_unknown_plugin_weighs_one(self):
|
|
assert _manager({})._plugin_weight('ghost') == 1
|
|
|
|
|
|
class TestTheSchedule:
|
|
def test_nothing_weighted_leaves_the_order_untouched(self):
|
|
order = ['weather', 'clock', 'news']
|
|
sm = _manager({p: FakePlugin() for p in order})
|
|
assert sm._apply_priority_weights(order) == order
|
|
|
|
def test_off_by_default_the_order_is_untouched(self):
|
|
order = ['weather', 'mlb', 'news']
|
|
sm = _manager({'weather': FakePlugin(), 'mlb': FakePlugin(live=True),
|
|
'news': FakePlugin()}, live_in_ticker=False, live_weight=3)
|
|
assert sm._apply_priority_weights(order) == order
|
|
|
|
def test_a_live_plugin_takes_its_share_of_slots(self):
|
|
order = ['weather', 'mlb', 'news', 'clock']
|
|
sm = _manager({'weather': FakePlugin(), 'mlb': FakePlugin(live=True),
|
|
'news': FakePlugin(), 'clock': FakePlugin()},
|
|
live_weight=3)
|
|
schedule = sm._apply_priority_weights(order)
|
|
counts = _counts(schedule)
|
|
assert counts['mlb'] == 3, counts
|
|
assert counts['weather'] == counts['news'] == counts['clock'] == 1, counts
|
|
assert len(schedule) == 6
|
|
|
|
def test_every_plugin_still_appears(self):
|
|
# A boost must not starve anything out of the cycle.
|
|
order = ['a', 'b', 'c', 'd', 'e', 'f']
|
|
plugins = {p: FakePlugin() for p in order}
|
|
plugins['a'] = FakePlugin(live=True, declared=10)
|
|
sm = _manager(plugins)
|
|
schedule = sm._apply_priority_weights(order)
|
|
assert set(schedule) == set(order), set(order) - set(schedule)
|
|
|
|
def test_nothing_doubles_across_the_cycle_seam(self):
|
|
# The strip loops, so the last slot neighbours the first. Smooth
|
|
# Weighted Round-Robin schedules the heaviest item first and often
|
|
# last too, which put the one clump the algorithm exists to avoid at
|
|
# the one place a within-cycle check cannot see.
|
|
order = ['baseball', 'weather', 'geochron', 'flights', 'stocks',
|
|
'oftheday', 'youtube', 'stocknews', 'leaderboard',
|
|
'countdown', 'odds', 'f1', 'football', 'music']
|
|
plugins = {p: FakePlugin() for p in order}
|
|
plugins['baseball'] = FakePlugin(live=True, declared=5)
|
|
plugins['football'] = FakePlugin(live=True, declared=3)
|
|
schedule = _manager(plugins)._apply_priority_weights(order)
|
|
|
|
n = len(schedule)
|
|
doubles = [schedule[i] for i in range(n)
|
|
if schedule[i] == schedule[(i + 1) % n]]
|
|
assert not doubles, "%r repeats across the seam in %r" % (doubles, schedule)
|
|
|
|
def test_the_seam_repair_keeps_every_slot(self):
|
|
order = ['a', 'b', 'c', 'd', 'e', 'f']
|
|
plugins = {p: FakePlugin() for p in order}
|
|
plugins['a'] = FakePlugin(live=True, declared=4)
|
|
schedule = _manager(plugins)._apply_priority_weights(order)
|
|
assert _counts(schedule)['a'] == 4, _counts(schedule)
|
|
assert sorted(schedule) == sorted(
|
|
['a'] * 4 + ['b', 'c', 'd', 'e', 'f']), schedule
|
|
|
|
def test_the_repair_uses_the_widest_gap(self):
|
|
# Moving the trailing repeat into the first slot that merely fits
|
|
# undoes the spacing: on a 28-slot rotation that turned a gap of 7
|
|
# into a gap of 2, which is more clumped than the seam ever was.
|
|
order = ['a'] + ['p%d' % i for i in range(13)]
|
|
plugins = {p: FakePlugin() for p in order}
|
|
plugins['a'] = FakePlugin(live=True, declared=4)
|
|
schedule = _manager(plugins)._apply_priority_weights(order)
|
|
at = [i for i, p in enumerate(schedule) if p == 'a']
|
|
gaps = [b - a for a, b in zip(at, at[1:])]
|
|
gaps.append(len(schedule) - at[-1] + at[0])
|
|
ideal = len(schedule) / len(at)
|
|
assert min(gaps) >= ideal / 2, "gaps %r for ideal %.1f" % (gaps, ideal)
|
|
|
|
def test_an_unavoidable_double_is_left_alone(self):
|
|
# Five of seven slots are the same plugin, so it must neighbour
|
|
# itself. Better to schedule it than to refuse or loop forever.
|
|
order = ['a', 'b', 'c']
|
|
plugins = {p: FakePlugin() for p in order}
|
|
plugins['a'] = FakePlugin(live=True, declared=5)
|
|
schedule = _manager(plugins)._apply_priority_weights(order)
|
|
assert _counts(schedule) == {'a': 5, 'b': 1, 'c': 1}, _counts(schedule)
|
|
assert set(schedule) == {'a', 'b', 'c'}
|
|
|
|
def test_the_repair_never_creates_a_new_double(self):
|
|
# The first version guarded the slot the repeated value moves *into*
|
|
# but not the one the displaced element lands in, so this traded the
|
|
# seam duplicate for a fresh one and came back ending ['x', 'x'].
|
|
sm = _manager({})
|
|
out = sm._unclump_seam(['a', 'b', 'c', 'd', 'x', 'y', 'x', 'a'])
|
|
n = len(out)
|
|
doubles = [out[i] for i in range(n) if out[i] == out[(i + 1) % n]]
|
|
assert not doubles, "%r in %r" % (doubles, out)
|
|
assert sorted(out) == sorted(['a', 'b', 'c', 'd', 'x', 'y', 'x', 'a'])
|
|
|
|
def test_the_last_two_slots_are_a_usable_swap(self):
|
|
# Reasoning about indices said this candidate was unsafe because
|
|
# schedule[j] is schedule[-2]; after the swap its neighbour is the
|
|
# repeated value, not itself. Refusing it left the only repair this
|
|
# schedule has on the table.
|
|
assert _manager({})._unclump_seam(['a', 'b', 'c', 'a']) == ['a', 'b', 'a', 'c']
|
|
|
|
def test_no_seam_schedule_is_ever_made_worse(self):
|
|
import random
|
|
sm = _manager({})
|
|
random.seed(11)
|
|
checked = 0
|
|
for size in range(3, 10):
|
|
for _ in range(400):
|
|
original = [random.choice('abcd') for _ in range(size)]
|
|
if original[0] != original[-1]:
|
|
continue
|
|
checked += 1
|
|
out = sm._unclump_seam(list(original))
|
|
n = len(out)
|
|
before = sum(1 for i in range(n)
|
|
if original[i] == original[(i + 1) % n])
|
|
after = sum(1 for i in range(n) if out[i] == out[(i + 1) % n])
|
|
assert after <= before, (original, out)
|
|
assert sorted(out) == sorted(original), (original, out)
|
|
assert checked > 100, "the generator stopped producing seam cases"
|
|
|
|
def test_a_schedule_too_short_to_repair_is_returned_as_is(self):
|
|
sm = _manager({})
|
|
assert sm._unclump_seam(['a', 'a']) == ['a', 'a']
|
|
assert sm._unclump_seam(['a']) == ['a']
|
|
assert sm._unclump_seam([]) == []
|
|
|
|
def test_a_schedule_with_no_seam_clash_is_untouched(self):
|
|
sm = _manager({})
|
|
plain = ['a', 'b', 'c', 'a', 'd']
|
|
assert sm._unclump_seam(plain) == plain
|
|
|
|
def test_repeats_are_spread_not_clumped(self):
|
|
# The point of Smooth Weighted Round-Robin. Three-in-a-row followed by
|
|
# a long silence would be worse than not boosting at all.
|
|
order = ['weather', 'mlb', 'news', 'clock', 'stocks', 'f1']
|
|
plugins = {p: FakePlugin() for p in order}
|
|
plugins['mlb'] = FakePlugin(live=True)
|
|
sm = _manager(plugins, live_weight=3)
|
|
schedule = sm._apply_priority_weights(order)
|
|
|
|
assert _counts(schedule)['mlb'] == 3
|
|
# Evenly spread over 8 slots means a gap of about 3, never 6.
|
|
assert _max_gap(schedule, 'mlb') <= 4, schedule
|
|
# And never twice running.
|
|
assert not any(a == b == 'mlb' for a, b in zip(schedule, schedule[1:])), schedule
|
|
|
|
def test_a_favorite_outranks_another_live_game(self):
|
|
order = ['weather', 'mlb', 'nhl']
|
|
sm = _manager({'weather': FakePlugin(),
|
|
'mlb': FakePlugin(live=True, declared=5),
|
|
'nhl': FakePlugin(live=True)}, live_weight=2)
|
|
counts = _counts(sm._apply_priority_weights(order))
|
|
assert counts['mlb'] == 5 and counts['nhl'] == 2 and counts['weather'] == 1, counts
|
|
|
|
def test_an_empty_rotation_is_harmless(self):
|
|
assert _manager({})._apply_priority_weights([]) == []
|
|
|
|
|
|
class TestConfigParsing:
|
|
def test_live_games_stay_in_the_ticker_by_default(self):
|
|
cfg = VegasModeConfig.from_config({})
|
|
assert cfg.live_in_ticker is True
|
|
assert cfg.live_weight == 3 and cfg.favorite_live_weight == 5
|
|
|
|
@pytest.mark.parametrize("given,expected", [(0, 1), (-4, 1), (99, 10), (4, 4)])
|
|
def test_weights_are_clamped(self, given, expected):
|
|
cfg = VegasModeConfig.from_config(
|
|
{'display': {'vegas_scroll': {'live_weight': given}}})
|
|
assert cfg.live_weight == expected
|