adafruit / lpd8806 Goto Github PK

Just a reminder to review the comments posted at http://forums.adafruit.com/viewtopic.php?f=47&t=23087
The clock pulse appears to light the first green LED. This happens even with the original library, although it is dimmer. Note that I don't think this is a software issue. I can produce the effect with the minimal sketch posted in the forum (above).

Just a small issue - the README says:
Place the LPD8806 library folder your <arduinosketchfolder>/libraries/ folder,

I think the word "in" is missing before "your," and also the <arduinosketchfolder> part isn't showing up in the markdown - you have to escape the < character.

Thank you for making this :)

I'm getting these compile errors when trying to upload the LEDBeltKit_alt sketch to Teensy 3.

FWIW - The strandtest and LEDBeltKit sketches works normally.

LEDbeltKit_alt.pde: In function 'void callback()':
LEDbeltKit_alt.pde:162:62: error: call of overloaded 'gamma(int)' is ambiguous
LEDbeltKit_alt.pde:162:62: note: candidates are:
In file included from /Applications/Arduino.app/Contents/Resources/Java/hardware/teensy/cores/teensy3/WProgram.h:6:0,
from /Applications/Arduino.app/Contents/Resources/Java/hardware/teensy/cores/teensy3/Arduino.h:1,
from /Applications/Arduino.app/Contents/Resources/Java/libraries/SPI/SPI.h:15,
from LEDbeltKit_alt.pde:20:
/Applications/Arduino.app/Contents/Resources/Java/hardware/tools/arm-none-eabi/bin/../lib/gcc/arm-none-eabi/4.7.2/include-fixed/math.h:287:15: note: double gamma(double)
LEDbeltKit_alt.pde:82:6: note: byte gamma(byte)
LEDbeltKit_alt.pde:163:62: error: call of overloaded 'gamma(int)' is ambiguous
LEDbeltKit_alt.pde:163:62: note: candidates are:
In file included from /Applications/Arduino.app/Contents/Resources/Java/hardware/teensy/cores/teensy3/WProgram.h:6:0,
from /Applications/Arduino.app/Contents/Resources/Java/hardware/teensy/cores/teensy3/Arduino.h:1,
from /Applications/Arduino.app/Contents/Resources/Java/libraries/SPI/SPI.h:15,
from LEDbeltKit_alt.pde:20:
/Applications/Arduino.app/Contents/Resources/Java/hardware/tools/arm-none-eabi/bin/../lib/gcc/arm-none-eabi/4.7.2/include-fixed/math.h:287:15: note: double gamma(double)
LEDbeltKit_alt.pde:82:6: note: byte gamma(byte)
LEDbeltKit_alt.pde:164:62: error: call of overloaded 'gamma(int)' is ambiguous
LEDbeltKit_alt.pde:164:62: note: candidates are:
In file included from /Applications/Arduino.app/Contents/Resources/Java/hardware/teensy/cores/teensy3/WProgram.h:6:0,
from /Applications/Arduino.app/Contents/Resources/Java/hardware/teensy/cores/teensy3/Arduino.h:1,
from /Applications/Arduino.app/Contents/Resources/Java/libraries/SPI/SPI.h:15,
from LEDbeltKit_alt.pde:20:
/Applications/Arduino.app/Contents/Resources/Java/hardware/tools/arm-none-eabi/bin/../lib/gcc/arm-none-eabi/4.7.2/include-fixed/math.h:287:15: note: double gamma(double)
LEDbeltKit_alt.pde:82:6: note: byte gamma(byte)

Compiling for an Intel Edison board throws a "F_CPU was not declared in this scope" error, in line 201 of LPD8806.cpp

It may be that F_CPU is not defined for Intel Edison boards?

Would it be possible to wrap a

#ifndef __LPD8806_H__
#define __LPD8806_H__

[class]

#endif

around the class definition in the .h file so one can include the .h file multiple times in one project?

I've just been playing with a new light strip using the example from here, I've put a slightly more funky one with animated rainbows up at https://gist.github.com/tomoinn/5969009 if that's of any interest.

In the setPixelColor method, should we check for n >= 0, in addition to the n < numLEDs.

Per request, a link to the hardware-SPI fork of this library:
https://github.com/cjbaar/LPD8806
--CJ

Hi
May I know how many LED does the library can drive
seems the Arduino would freeze when more than 160 LEDs
thanks

It would be nice if the library support code like this:

strip.setPixelColorRGB(0, 0xFF0000);  // RED
strip.setPixelColorRGB(1, 0x00FF00);  // GREEN
strip.setPixelColorRGB(2, 0x0000FF);  // BLUE
strip.setPixelColorRGB(3, 0xFFFFFF);  // WHITE

Had problems with this line using this library in PlatformIO

depends=TimerOne

My project platformio.ini config:
[env:esp32doit-devkit-v1]
platform = espressif32
board = esp32doit-devkit-v1
framework = arduino

It seems that TimerOne doesn't compile in this config. I dropped back to 1.0.1 without this line and all is good.
Just FYI
Bleep,