qmk/qmk_firmware

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

Open

#25.406 geöffnet am 24. Juni 2025

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C (43.867 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (20.368 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 20T 9h) (27 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die Änderungen in PR #24364, die die rechte Indexgrenzprüfung entfernt haben. Finden Sie die fehlende Bedingung im WS2812 RGB Matrix Code und fügen Sie sie wieder hinzu, um Indizes außerhalb des Bereichs der Hälfte zu ignorieren.
Tech Stack
c
Domain
embedded iot
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
CGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60