qmk/qmk_firmware

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

Ouverte

#25 406 ouverte le 24 juin 2025

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C (43 867 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (20 368 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Inspectez les modifications apportées à la PR #24364 qui ont supprimé la vérification de la limite d'index droit. Localisez la condition manquante dans le code de la matrice RVB WS2812 et rajoutez la pour ignorer les index en dehors de la plage de la moitié.
Stack technique
c
Domaine
embedded iot
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60