1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

FIXED Testers wanted: APA102 bug fix

Discussion in 'Bug Reports' started by penfold42, 2 May 2016.

  1. penfold42

    penfold42 Moderator Developer

    Messages:
    750
    Hardware:
    RPi1/Zero, RPi2, RPi3, 32/64bit, +Arduino, +nodeMCU/ESP8266
    • Like Like x 3
  2. Brindosch

    Brindosch Active Member

    Messages:
    678
    Hardware:
    RPi1/Zero, RPi2, RPi3, +nodeMCU/ESP8266
    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 off

    No changes to the current binary

    Thank you for your efforts.
     
  3. penfold42

    penfold42 Moderator Developer

    Messages:
    750
    Hardware:
    RPi1/Zero, RPi2, RPi3, 32/64bit, +Arduino, +nodeMCU/ESP8266
    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
     
  4. Brindosch

    Brindosch Active Member

    Messages:
    678
    Hardware:
    RPi1/Zero, RPi2, RPi3, +nodeMCU/ESP8266
    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 :)
     
  5. penfold42

    penfold42 Moderator Developer

    Messages:
    750
    Hardware:
    RPi1/Zero, RPi2, RPi3, 32/64bit, +Arduino, +nodeMCU/ESP8266
    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 ?
     
  6. Brindosch

    Brindosch Active Member

    Messages:
    678
    Hardware:
    RPi1/Zero, RPi2, RPi3, +nodeMCU/ESP8266
    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.
     
  7. penfold42

    penfold42 Moderator Developer

    Messages:
    750
    Hardware:
    RPi1/Zero, RPi2, RPi3, 32/64bit, +Arduino, +nodeMCU/ESP8266
    In an idiot..

    Fix for the fix coming soon

    Thanks
     
  8. penfold42

    penfold42 Moderator Developer

    Messages:
    750
    Hardware:
    RPi1/Zero, RPi2, RPi3, 32/64bit, +Arduino, +nodeMCU/ESP8266
    I've pushed an updated fix with updated binary.

    Could you pls test again ?
     
    • Like Like x 1
  9. Brindosch

    Brindosch Active Member

    Messages:
    678
    Hardware:
    RPi1/Zero, RPi2, RPi3, +nodeMCU/ESP8266
  10. apurbas

    apurbas New Member

    Messages:
    6
    Hardware:
    RPi1/Zero, RPi2, +PhilipsHue
  11. Brindosch

    Brindosch Active Member

    Messages:
    678
    Hardware:
    RPi1/Zero, RPi2, RPi3, +nodeMCU/ESP8266
  12. redPanther

    redPanther Moderator Developer

    Messages:
    211
    Hardware:
    RPi1/Zero, RPi2, 32/64bit, +Arduino
    I have a works for all led devices solution. Stay tuned ;-) I have to do some cleanup and refactoring task before merge