From 40a740f7e43405c661b9c44aebd7bfac9ac9b2bb Mon Sep 17 00:00:00 2001
From: jim kolpack <james.kolpack@gmail.com>
Date: Fri, 9 Oct 2026 18:51:25 -0400
Subject: [PATCH] Shorten RGB-split bands to 4px max, per request

GLITCH_STATE_POOL's height values were 10-13px (sized to reliably show
2-3 scanlines within a band); capped to 2-4px thin slivers instead. No
longer tall enough to guarantee scanline overlap, which is fine -- the
shorter height was the explicit ask.

Verified 0 black-corner hits across 228 frames, payload 291KB (down
from 320KB with the taller bands).
---
 subwave_nowplaying.star | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/subwave_nowplaying.star b/subwave_nowplaying.star
index 0cd0397..e89e588 100644
--- a/subwave_nowplaying.star
+++ b/subwave_nowplaying.star
@@ -393,19 +393,19 @@ GLITCH_STATES_PER_CYCLE = 4
 # height) -- see the note above GLITCH_HOLD_TICKS for why that's the only
 # part grounded in anything observed rather than invented.
 GLITCH_STATE_POOL = [
-    # Scanlines (see _glitch_scanlines below) repeat every 4px, so each
-    # band's height needs to be at least ~9-13px to reliably show 2-3 of
-    # them, regardless of the band's exact vertical alignment against the
-    # scanline grid -- much shorter and a band can land entirely between
-    # two scanline rows and show none at all.
-    (3, 12, -2, 150.0, 19, 11, 2, -60.0),
-    (2, 11, 2, 150.0, 20, 12, -3, -60.0),
-    (5, 10, -3, 150.0, 17, 13, 1, -60.0),
-    (1, 13, 1, 150.0, 21, 10, -2, -60.0),
-    (6, 10, 1, 150.0, 14, 12, 3, -60.0),
-    (0, 11, 3, 150.0, 20, 10, -1, -60.0),
-    (4, 10, -1, 150.0, 12, 13, 2, -60.0),
-    (2, 12, -3, 150.0, 22, 10, 1, -60.0),
+    # Capped to 4px tall -- thin RGB-split slivers rather than the
+    # original 10-13px bands. No longer tall enough to reliably show 2-3
+    # scanlines within a given band (that required ~9-13px against
+    # _glitch_scanlines' 4px repeat), so a band can land entirely between
+    # two scanline rows; accepted, since the shorter height was the ask.
+    (3, 3, -2, 150.0, 18, 4, 2, -60.0),
+    (2, 4, 2, 150.0, 20, 3, -3, -60.0),
+    (5, 2, -3, 150.0, 16, 4, 1, -60.0),
+    (1, 4, 1, 150.0, 22, 2, -2, -60.0),
+    (6, 3, 1, 150.0, 14, 3, 3, -60.0),
+    (0, 2, 3, 150.0, 24, 4, -1, -60.0),
+    (4, 3, -1, 150.0, 19, 2, 1, -60.0),
+    (2, 4, -3, 150.0, 26, 3, 1, -60.0),
 ]
 
 # render.Sequence doesn't loop on its own (see _glitch_scanlines below), so