-
Notifications
You must be signed in to change notification settings - Fork 38
Support other color orders like NeoPixel #15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I'm not sure these will need any changes. A forum user (may be @RedAnon ) is going to test next week and get back to me. |
Mimic the NeoPixel library. https://github.com/adafruit/Adafruit_CircuitPython_NeoPixel/blob/master/neopixel.py#L39 |
Thanks. I've already forwarded this to our company's developer and he will try to implement it. |
Mine! Working on this @ pycon sprints |
What's your ETA @mcscope? I can't promise you we won't be faster. :P |
@RedAnon I'm working on it now @ the pycon sprints. |
other issue about pixel brightness value: #16 |
@mcscope Yes it was. AFAIK the driver right now simply sends a fixed value but people from APA USA told me that this should only work for their 8-bit LEDs (even tho it plainly works for the 13-bit ones as well as I've demonstrated on the Adafruit forums). |
@RedAnon I'm not seeing anything about a 13 bit version of this strip anywhere. Are you adding the 5 bit brightness value to one of the 8 bit color values to get 13? |
PR: #18 |
@mcscope And yes, as far as I know the 5-Bit value added to the 8-Bit colour plays an important role in fully utilizing the 13-Bit chips. Problem is the data from the manufacturer are a bit sketchy. For example they were adamant that sending 8-Bit code to 13-Bit LEDs would never ever work but it does... |
Fixed by #18 |
APA Electronic has a new 13-bit version of the APA-102 LEDs which have a lower refresh rate (approximately 700MHz) but smoother colours.
As far as I can tell there is currently no way of controlling these chips. And it would be extremely awesome if we could control them.
The text was updated successfully, but these errors were encountered: