Skip to content

Conversation

@makermelissa
Copy link
Collaborator

Fixes #9. Tested with:

import board
import adafruit_ws2801

odata = board.D5
oclock = board.D6
numleds = 25
bright = 1.0
leds = adafruit_ws2801.WS2801(oclock, odata, numleds, brightness=bright, auto_write=True)
>>> leds[0] = (255, 0, 0)
>>> leds[0]
(255, 0, 0)
>>> leds[0:3] = ((255, 0, 0), (0, 255, 0), (0, 0, 255))
>>> leds[0:3]
[(255, 0, 0), (0, 255, 0), (0, 0, 255)]

@makermelissa makermelissa requested a review from a team November 5, 2019 00:47
@makermelissa makermelissa merged commit 2ed038d into adafruit:master Nov 5, 2019
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Nov 5, 2019
@kevinjwalters
Copy link

Looking back at how this happened, I would have based this off code like https://github.com/adafruit/Adafruit_CircuitPython_DotStar/blob/7997057de145b9900426379772f46718bd563529/adafruit_dotstar.py which uses a B,G,R order, hence the reversal.

I changed that to R,G,B for the ordering required for the WS2801 LEDs I had but must have not tested reading back the values!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Value returned is backwards of what it was set to

3 participants