tinue / apa102-pi

Pure Python library to drive APA102 LED stripes; Use with Raspberry Pi.
GNU General Public License v2.0
202 stars 72 forks source link

setPixel bug? #4

Closed psycholein closed 8 years ago

psycholein commented 8 years ago

https://github.com/tinue/APA102_Pi/blob/master/apa102.py#L123 i have to swap green and blue here?!

hint: u can precalculate the endFrame in the constructor, so u don't need a loop there

tinue commented 8 years ago

I should add a way too configure the strip coloring. In my strip the colors are correct, but there are other APA102 variants out there. At the moment, you have to swap.

tinue commented 8 years ago

Good idea about the end frame pre-calculation, I'll add this in the next revision. It will take a while, though: I don't have much time for the project at the moment.

tinue commented 8 years ago

Closing: Fix from kapacuk solves the issue.