Testers wanted: APA102 bug fix
- penfold42
- Erledigt
-
-
I tested your test bin.
I deleted the 3 last leds of my .json (119,120,121)
result:
119 still light up with a "random static color"
120 still light up with a "random static color"
121 is offNo changes to the current binary
Thank you for your efforts.
-
You're holding it wrong...
Actually, I didn't document how to hold it.
Add "leds" : 122 to the device configuration to tell it how many physical LEDs you have.
The code needs to know how much trailing data to write to silence the "extra" LEDs
-
If i place the leds 119 (the real count, the problem with the 2 pixels persist)
If i place the total led count 122 (config 119) all pixes after led nr 119 light up in a bright white.So, no. But i have to admit i don´t need this feature, all for apurbas now
-
Thanks.... Hmmmm ...
Just so I'm clear, you have 122 physical LEDs (indexes 0..121) ?
If you comment out indexes 119..121 and set LEDs to 122 my fix does work but sets the extra LEDs to white instead of black ?
-
yes 122 real leds.
if i set leds to 119 and 119 are assigned at the config. nothing changes (expected)
If i set leds to 122 and 119 are assigned at the config all 3 leds shine bright instead of black. -
In an idiot..
Fix for the fix coming soon
Thanks
-
I've pushed an updated fix with updated binary.
Could you pls test again ?
-
//Fixed
you could submit a pull request
@apurbas thanks to @penfold42 and me -
Thainks @Brindosch & @penfold42 ..I was away for a while. You guys are awesome !
-
Yeah @penfold42 really is
-
I have a works for all led devices solution. Stay tuned I have to do some cleanup and refactoring task before merge
Jetzt mitmachen!
Sie haben noch kein Benutzerkonto auf unserer Seite? Registrieren Sie sich kostenlos und nehmen Sie an unserer Community teil!