qmk/qmk_firmware

[Bug] `RGB_MATRIX_SPLIT` + WS2812 behaves wrong on right side

Open

#25.406 aperta il 24 giu 2025

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C (43.867 fork)batch import
bughelp wanted

Metriche repository

Star
 (20.368 star)
Metriche merge PR
 (Merge medio 20g 9h) (27 PR mergiate in 30 g)

Descrizione

Describe the Bug

As diagnosed a few weeks ago (forgot to open this before) at discord

Culprit seems to be that #24364 removed/broke the right-side check for "if index is not within this half's boundaries, do nothing".

This is, with "split_count": [X, Y]:

  • Using an index [0, X) on left side should affect those LEDs: works
  • Using an index [X, X+Y) on right side should affect those LEDs: works
  • Using an index [X, X+Y) on left side should be ignored: works
  • Using an index [0, X) on right side should be ignored: doesn't work, it affects leds [X, X+X) AKA first X leds on the half's chain

Keyboard Used

No response

Link to product page (if applicable)

No response

Operating System

No response

qmk doctor Output

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

No response

Guida contributor

Direzione di ricerca
Esamina le modifiche nella PR #24364 che hanno rimosso il controllo dei confini dell'indice destro. Individua la condizione mancante nel codice della matrice RGB WS2812 e riaggiungila per ignorare gli indici al di fuori del range della metà.
Tech stack
c
Dominio
embedded iot
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60