Converted alt_rgb_pins to color_order

Author: makermelissa

adafruit_matrixportal/graphics.py

@@ -42,7 +42,8 @@ class Graphics:
     :param int height: The height of the display in Pixels. Defaults to 32.
     :param int bit_depth: The number of bits per color channel. Defaults to 2.
     :param list alt_addr_pins: An alternate set of address pins to use. Defaults to None
-    :param list alt_rgb_pins: An alternate set of rgb pins to use. Defaults to None
+    :param string color_order: A string containing the letter "R", "G", and "B" in the
+                               order you want. Defaults to "RGB"
     :param debug: Turn on debug print outs. Defaults to False.
 
     """
@@ -56,7 +57,7 @@ def __init__(
         height=32,
         bit_depth=2,
         alt_addr_pins=None,
-        alt_rgb_pins=None,
+        color_order="RGB",
         debug=False
     ):
 
@@ -66,7 +67,7 @@ def __init__(
             width=width,
             height=height,
             alt_addr_pins=alt_addr_pins,
-            alt_rgb_pins=alt_rgb_pins,
+            color_order=color_order,
         )
         self.display = matrix.display
 

adafruit_matrixportal/matrix.py

@@ -45,15 +45,25 @@ class Matrix:
     :param int height: The height of the display in Pixels. Defaults to 32.
     :param int bit_depth: The number of bits per color channel. Defaults to 2.
     :param list alt_addr_pins: An alternate set of address pins to use. Defaults to None
-    :param list alt_rgb_pins: An alternate set of rgb pins to use. Defaults to None
+    :param string color_order: A string containing the letter "R", "G", and "B" in the
+                               order you want. Defaults to "RGB"
 
     """
 
     # pylint: disable=too-few-public-methods,too-many-branches
     def __init__(
-        self, *, width=64, height=32, bit_depth=2, alt_addr_pins=None, alt_rgb_pins=None
+        self, *, width=64, height=32, bit_depth=2, alt_addr_pins=None, color_order="RGB"
     ):
 
+        if not isinstance(color_order, str):
+            raise ValueError("color_index should be a string")
+        color_order = color_order.lower()
+        red_index = color_order.find("r")
+        green_index = color_order.find("g")
+        blue_index = color_order.find("b")
+        if -1 in (red_index, green_index, blue_index):
+            raise ValueError("color_order should contain R, G, and B")
+
         if "Matrix Portal M4" in os.uname().machine:
             # MatrixPortal M4 Board
             addr_pins = [board.MTX_ADDRA, board.MTX_ADDRB, board.MTX_ADDRC]
@@ -108,10 +118,6 @@ def __init__(
             latch_pin = board.D10
             oe_pin = board.D9
 
-        # Alternate Address Pins
-        if alt_rgb_pins is not None:
-            rgb_pins = alt_rgb_pins
-
         # Alternate Address Pins
         if alt_addr_pins is not None:
             addr_pins = alt_addr_pins
@@ -122,7 +128,14 @@ def __init__(
                 width=width,
                 height=height,
                 bit_depth=bit_depth,
-                rgb_pins=rgb_pins,
+                rgb_pins=(
+                    rgb_pins[red_index],
+                    rgb_pins[green_index],
+                    rgb_pins[blue_index],
+                    rgb_pins[red_index + 3],
+                    rgb_pins[green_index + 3],
+                    rgb_pins[blue_index + 3],
+                ),
                 addr_pins=addr_pins,
                 clock_pin=clock_pin,
                 latch_pin=latch_pin,

adafruit_matrixportal/matrixportal.py

@@ -58,7 +58,8 @@ class MatrixPortal:
     :param busio.SPI external_spi: A previously declared spi object. Defaults to ``None``.
     :param int bit_depth: The number of bits per color channel. Defaults to 2.
     :param list alt_addr_pins: An alternate set of address pins to use. Defaults to None
-    :param list alt_rgb_pins: An alternate set of rgb pins to use. Defaults to None
+    :param string color_order: A string containing the letter "R", "G", and "B" in the
+                               order you want. Defaults to "RGB"
     :param debug: Turn on debug print outs. Defaults to False.
 
     """
@@ -78,7 +79,7 @@ def __init__(
         external_spi=None,
         bit_depth=2,
         alt_addr_pins=None,
-        alt_rgb_pins=None,
+        color_order="RGB",
         debug=False
     ):
 
@@ -89,7 +90,7 @@ def __init__(
             width=64,
             height=32,
             alt_addr_pins=alt_addr_pins,
-            alt_rgb_pins=alt_rgb_pins,
+            color_order=color_order,
             debug=debug,
         )
         self.display = self.graphics.display