adafruit / Adafruit_CircuitPython_DotStar

Dotstarrin' away in CircuitPython land
MIT License
46 stars 38 forks source link

Dotstar updates are slow #25

Closed rhooper closed 4 years ago

rhooper commented 6 years ago

The dotstar library could benefit from a CircuitPython helper to offload computationally expensive setitem work to. See adafruit/circuitpython#884

rhooper commented 6 years ago

I have done some experimental work on this for Neopixels, see adafruit/circuitpython#884 and https://github.com/adafruit/Adafruit_CircuitPython_NeoPixel/issues/27

kattni commented 4 years ago

@tannewt This needs to be updated to work with pixelbuf as well. There is some work here: https://github.com/rhooper/Adafruit_CircuitPython_DotStar/blob/pixelbuf/adafruit_dotstar.py

tannewt commented 4 years ago

@dunkmann00 want to take a stab at it?

dunkmann00 commented 4 years ago

Sure! I don't have any dotstars to test it though. But I think it should be straight forward enough, especially since there is already some work done on it.

tannewt commented 4 years ago

@dunkmann00 That'd be awesome. We can find someone with DotStars to verify the code. Thanks!

kattni commented 4 years ago

We did it!