Discussion in 'Bug Reports' started by penfold42, 2 May 2016.
As per: https://github.com/hyperion-project/hyperion/issues/612
I tested your test bin.
I deleted the 3 last leds of my .json (119,120,121)
119 still light up with a "random static color"
120 still light up with a "random static color"
121 is off
No 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
I've pushed an updated fix with updated binary.
Could you pls test again ?
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
Separate names with a comma.