search

Lixie-Labs / Emotiscope

Let your eyes enjoy the music too ✨ Extremely low-latency audio processing lets LEDs perfectly synchronize to your music's notation and tempo.
GNU General Public License v3.0
27 stars 5 forks source link

Add support for NUM_LEDS other than 128 #79

Closed larryqiann closed 1 month ago

larryqiann commented 2 months ago

fix_background_longer_leds.patch

Only tested on the 1.1 branch as of 79105ca currently but I don't think the files have changed much since 1.0. This patch solves [some of?] the problems with using Emotiscope with more or less than 128 LEDs as defined in leds.h/NUM_LEDS.

The main issue previously was that the background wouldn't extend to the full strip in mirror mode in many of the modes (non-mirrored modes worked fine). These patches attempt to remove hard coded references to 63/64, which are the lengths of the hard-coded 128 LED strip.

connornishijima commented 1 month ago

Working on this today!

connornishijima commented 1 month ago

Fixed in https://github.com/Lixie-Labs/Emotiscope/commit/b9b981b88dc7a8ecdc27867b3e82eb2d7d26e187!