Update the API to support pypixelbuf 2.0

[dunkmann00]

Voila! #14

[dunkmann00]

I'm pretty sure that both of those pylint complaints will go away with the changes to pypixelbuf.

[caternuson]

What if it's a string instead of a tuple?

Strings are currently used in this library, but I'm not sure what the current desire is for pixel_order? tuple? string? both?

[dunkmann00]

I should have left that line outside of the tuple check. That way when pixel_order is supplied we always use its length as bpp. Then we check if it is a tuple, if yes, it gets converted to a string. And if it is already a string it is good to go as is.

This fixes the problem you were seeing for RBGW as well.

[caternuson]

See code comment. Works for RGB but breaks for RGBW. It's failing to set bpp correctly.

import time
import board
import neopixel_spi as neopixel

NUM_PIXELS = 12
PIXEL_ORDER = neopixel.GRBW
COLORS = (0xFF0000, 0x00FF00, 0x0000FF)
DELAY = 0.1

pixels = neopixel.NeoPixel_SPI(board.SPI(), NUM_PIXELS, pixel_order=PIXEL_ORDER)

while True:
    for color in COLORS:
        for i in range(NUM_PIXELS):
            pixels[i] = color
            pixels.show()
            time.sleep(DELAY)
            pixels.fill(0)

Adafruit CircuitPython 5.1.0 on 2020-04-02; Adafruit ItsyBitsy M4 Express with samd51g19
>>> import test
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "test.py", line 15, in <module>
  File "/lib/neopixel_spi.py", line 147, in _transmit
  File "/lib/neopixel_spi.py", line 163, in _transmogrify
IndexError: bytearray index out of range
>>>  

[caternuson]

Yep. That seems to have fixed it.

[tannewt]

Ok great! I'm going to merge now because the lint errors are based on the old version of pypixelbuf that is released.