adding alternative TLC59711Multi driver implementation

.gitignore

@@ -1,11 +1,13 @@
 *.mpy
 .idea
+*.idea.py
 __pycache__
 _build
 *.pyc
 .env
+build*
 bundles
 *.DS_Store
 .eggs
 dist
-**/*.egg-info
+**/*.egg-info

.travis.yml

@@ -0,0 +1,33 @@
+dist: trusty
+sudo: false
+language: python
+python:
+- '3.6'
+cache:
+  pip: true
+deploy:
+- provider: releases
+  api_key: "$GITHUB_TOKEN"
+  file_glob: true
+  file: "$TRAVIS_BUILD_DIR/bundles/*"
+  skip_cleanup: true
+  overwrite: true
+  on:
+    tags: true
+- provider: pypi
+  user: adafruit-travis
+  on:
+    tags: true
+  password:
+    secure: Bb5O7KFF8KzwlRBmKuBD4MXOZfycKyHvsghpaGabp9ckZWy+wqS1kKlYbosXbRuAlF4xeGNMFRDBXDx6nEQFa70ozFqgqfA6CpCDtgQgtGhIQxthurR9Mrob4o9kCzwdG8rk1+XC35Ab290bnwLF27tGUm/qeOKhrK/WPOq1VBe4tKjF1HuPhvFljJFa10w1kST+EeSDOnJr+y5o4Cw8Vdoo++e86h184b0vXFT01nefpRDXtHDnzGT+MT24jnfdkAivKuTODHRih7OgpkiyzZKZgyf7oFkKdPqjA0XODTCmnKH9gzFZJ+aeqQUQt2bHk0+yxezy6BV6HRITpPkYW3hH/TCYSUDNexGx1rfGVWrMs0F9wN2lMWyGEzWH6wnBcXQWiCpnRbQOHEeUA4502TO3Ybk3Brr9Y8uFWAIJiIR3fzk0SmhPOT8GN/stmz7U5KiIaXTxeTJaEmZcokE/9NJtqORNXboM9sDqVcXkN59muibkFKN0Zsg+YPEiieitGQv4bPedmg1lW1Bl4G36Vf/xkX2P8XlDq2f8ADLPdQhcjLuQmxHfUHiWrKonqmEfr49E1DQ+fDqwkOCEG0MykQlN0q1MFiZNs6OPSwW5yjsGLMUsfWHqQXYc5ZXtAp5twRHHqPtdqAaKLSbN1v/a9cNO/8oqn+I4A5L9ImYtFS4=
+install:
+- pip install -r requirements.txt
+- pip install pylint circuitpython-build-tools Sphinx sphinx-rtd-theme
+- pip install --force-reinstall pylint==1.9.2
+script:
+- pylint adafruit_tlc59711/*.py
+- ([[ ! -d "examples" ]] || pylint --disable=missing-docstring,invalid-name,bad-whitespace
+  examples/*.py)
+- circuitpython-build-bundles --filename_prefix adafruit-circuitpython-tlc59711 --library_location
+  .
+- cd docs && sphinx-build -E -W -b html . _build/html && cd ..

adafruit_tlc59711.py → adafruit_tlc59711/adafruit_tlc59711.py

@@ -48,21 +48,21 @@
 # Globally disable invalid-name check as this chip by design has short channel
 # and register names.  It is confusing to rename these from what the datasheet
 # refers to them as.
-#pylint: disable=invalid-name
+# pylint: disable=invalid-name
 
 # Globally disable too many instance attributes check.  Again this is a case
 # where pylint doesn't have the right context to make this call.  The chip by
 # design has many channels which must be exposed.
-#pylint: disable=too-many-instance-attributes
+# pylint: disable=too-many-instance-attributes
 
 # Globally disable protected access.  Once again pylint can't figure out the
-# context for using internal decorate classes below.  In these cases protectected
-# access is by design for the internal class.
-#pylint: disable=protected-access
+# context for using internal decorate classes below.
+# In these cases protected access is by design for the internal class.
+# pylint: disable=protected-access
 
 # Yet another pylint issue, it fails to recognize a decorator class by
 # definition has no public methods.  Disable the check.
-#pylint: disable=too-few-public-methods
+# pylint: disable=too-few-public-methods
 
 
 def _shift_in(target_byte, val):
@@ -83,12 +83,13 @@ class TLC59711:
     independent channel by name (r0, g0, b0, r1, b1, etc.) as properties for
     fine-grained control.
 
-    :param ~busio.SPI spi: An instance of the SPI bus connected to the chip.  The clock and
-        MOSI/outout must be set, the MISO/input is unused.
-    :param bool auto_show: This is a boolean that defaults to True and indicates any
-        change to a channel value will instantly be written to the chip. You might wish to
-        set this to false if you desire to perform your own atomic operations of channel
-        values. In that case call the show function after making updates to channel state.
+    :param ~busio.SPI spi: An instance of the SPI bus connected to the chip.
+        The clock and MOSI/outout must be set, the MISO/input is unused.
+    :param bool auto_show: This is a boolean that defaults to True and
+        indicates any change to a channel value will instantly be written
+        to the chip. You might wish to set this to false if you desire
+        to perform your own atomic operations of channel values. In that case
+        call the show function after making updates to channel state.
     """
 
     class _GS_Value:
@@ -105,13 +106,13 @@ def __init__(self, byte_offset):
         def __get__(self, obj, obj_type):
             # Grab the 16-bit value at the offset for this channel.
             return (obj._shift_reg[self._byte_offset] << 8) | \
-                    obj._shift_reg[self._byte_offset+1]
+                obj._shift_reg[self._byte_offset + 1]
 
         def __set__(self, obj, val):
             # Set the 16-bit value at the offset for this channel.
             assert 0 <= val <= 65535
             obj._shift_reg[self._byte_offset] = (val >> 8) & 0xFF
-            obj._shift_reg[self._byte_offset+1] = val & 0xFF
+            obj._shift_reg[self._byte_offset + 1] = val & 0xFF
             # Write out the new values if auto_show is enabled.
             if obj.auto_show:
                 obj._write()
@@ -139,7 +140,7 @@ def __set__(self, obj, val):
     r0 = _GS_Value(26)
 
 
-    def __init__(self, spi, *, auto_show=True):
+    def __init__(self, spi, *, auto_show=True):  # noqa
         self._spi = spi
         # This device is just a big 28 byte long shift register without any
         # fancy update protocol.  Blast out all the bits to update, that's it!
@@ -220,8 +221,9 @@ def show(self):
     # Define properties for global brightness control channels.
     @property
     def red_brightness(self):
-        """The red brightness for all channels (i.e. R0, R1, R2, and R3).  This is a 7-bit
-           value from 0-127.
+        """The red brightness for all channels (i.e. R0, R1, R2, and R3).
+
+        This is a 7-bit value from 0-127.
         """
         return self._bcr
 
@@ -234,8 +236,9 @@ def red_brightness(self, val):
 
     @property
     def green_brightness(self):
-        """The green brightness for all channels (i.e. G0, G1, G2, and G3).  This is a
-           7-bit value from 0-127.
+        """The green brightness for all channels (i.e. G0, G1, G2, and G3).
+
+        This is a 7-bit value from 0-127.
         """
         return self._bcg
 
@@ -248,8 +251,9 @@ def green_brightness(self, val):
 
     @property
     def blue_brightness(self):
-        """The blue brightness for all channels (i.e. B0, B1, B2, and B3).  This is a 7-bit
-           value from 0-127.
+        """The blue brightness for all channels (i.e. B0, B1, B2, and B3).
+
+        This is a 7-bit value from 0-127.
         """
         return self._bcb
 
@@ -287,9 +291,11 @@ def __setitem__(self, key, val):
         """Set the R, G, B values for the provided channel.  Specify a
            3-tuple of R, G, B values that are each 16-bit numbers (0-65535).
         """
-        assert 0 <= key <= 3  # Do this check here instead of later to
-                              # prevent accidentally keeping auto_show
-                              # turned off when a bad key is provided.
+        # Do this check here instead of later to
+        # prevent accidentally keeping auto_show
+        # turned off when a bad key is provided.
+        assert 0 <= key <= 3
+
         assert len(val) == 3
         assert 0 <= val[0] <= 65535
         assert 0 <= val[1] <= 65535