linted

Author: ladyada

adafruit_focaltouch.py

@@ -45,16 +45,16 @@
 
 _FT6206_DEFAULT_I2C_ADDR = 0x38
 
-_FT6XXX_REG_DATA = 0x00
-_FT6XXX_REG_NUMTOUCHES = 0x02
-_FT6XXX_REG_THRESHHOLD = 0x80
-_FT6XXX_REG_POINTRATE = 0x88
-_FT6XXX_REG_LIBH = 0xA1
-_FT6XXX_REG_LIBL = 0xA2
-_FT6XXX_REG_CHIPID = 0xA3
-_FT6XXX_REG_FIRMVERS = 0xA6
-_FT6XXX_REG_VENDID = 0xA8
-_FT6XXX_REG_RELEASE = 0xAF
+_FT6XXX_REG_DATA = const(0x00)
+_FT6XXX_REG_NUMTOUCHES = const(0x02)
+_FT6XXX_REG_THRESHHOLD = const(0x80)
+_FT6XXX_REG_POINTRATE = const(0x88)
+_FT6XXX_REG_LIBH = const(0xA1)
+_FT6XXX_REG_LIBL = const(0xA2)
+_FT6XXX_REG_CHIPID = const(0xA3)
+_FT6XXX_REG_FIRMVERS = const(0xA6)
+_FT6XXX_REG_VENDID = const(0xA8)
+_FT6XXX_REG_RELEASE = const(0xAF)
 
 class Adafruit_FocalTouch:
     """
@@ -63,41 +63,46 @@ class Adafruit_FocalTouch:
 
     _debug = False
     chip = None
-    
+
     def __init__(self, i2c, address=_FT6206_DEFAULT_I2C_ADDR, debug=False):
         self._i2c = I2CDevice(i2c, address)
         self._debug = debug
 
         chip_data = self._read(_FT6XXX_REG_LIBH, 9)
-        lib_ver, chip_id, g_mode, pwr_mode, firm_id, skip, vend_id = struct.unpack('>HBBBBBB', chip_data)
+        lib_ver, chip_id, _, _, firm_id, _, vend_id = struct.unpack('>HBBBBBB', chip_data)
 
         if vend_id != 0x11:
             raise RuntimeError("Did not find FT chip")
 
         if chip_id == 0x06:
-            chip = "FT6206"
+            self.chip = "FT6206"
         elif chip_id == 0x64:
-            chip = "FT6236"
+            self.chip = "FT6236"
 
         if debug:
             print("Library vers %04X" % lib_ver)
             print("Firmware ID %02X" % firm_id)
             print("Point rate %d Hz" % self._read(_FT6XXX_REG_POINTRATE, 1)[0])
             print("Thresh %d" % self._read(_FT6XXX_REG_THRESHHOLD, 1)[0])
-            
-
 
     @property
     def touched(self):
-         return self._read(_FT6XXX_REG_NUMTOUCHES, 1)[0]
+        """ Returns the number of touches currently detected """
+        return self._read(_FT6XXX_REG_NUMTOUCHES, 1)[0]
 
+
+    # pylint: disable=unused-variable
     @property
     def touches(self):
+        """
+        Returns a list of touchpoint dicts, with 'x' and 'y' containing the
+        touch coordinates, and 'id' as the touch # for multitouch tracking
+        """
         touchpoints = []
         data = self._read(_FT6XXX_REG_DATA, 32)
 
-        for t in range(2):
-            point_data = data[t*6+3 : t*6+9]
+        for i in range(2):
+            point_data = data[i*6+3 : i*6+9]
             if all([i == 0xFF for i in point_data]):
                 continue
             #print([hex(i) for i in point_data])
@@ -106,9 +111,10 @@ def touches(self):
             touch_id = y >> 12
             x &= 0xFFF
             y &= 0xFFF
-            point = {'x':x , 'y': y, 'id': touch_id}
+            point = {'x':x, 'y':y, 'id':touch_id}
             touchpoints.append(point)
         return touchpoints
+    # pylint: enable=unused-variable
 
     def _read(self, register, length):
         """Returns an array of 'length' bytes from the 'register'"""
@@ -127,4 +133,3 @@ def _write(self, register, values):
             i2c.write(bytes(values))
             if self._debug:
                 print("\t$%02X <= %s" % (values[0], [hex(i) for i in values[1:]]))
-

examples/print_touches.py

@@ -1,4 +1,8 @@
-import time
+"""
+Example for getting touch data from an FT6206 or FT6236 capacitive
+touch driver, over I2C
+"""
+
 import busio
 import board
 import adafruit_focaltouch

examples/simple_paint.py

@@ -1,11 +1,14 @@
-# Simple painting demo
+"""
+Simple painting demo that draws on an Adafruit capacitive touch shield with
+ILI9341 display and FT6206 captouch driver
+"""
 
-import time
 import busio
 import board
 import digitalio
 import adafruit_focaltouch
-     
+from adafruit_rgb_display import ili9341, color565
+
 # Create library object using our Bus I2C & SPI port
 i2c = busio.I2C(board.SCL, board.SDA)
 spi = busio.SPI(clock=board.SCK, MOSI=board.MOSI, MISO=board.MISO)
@@ -15,7 +18,6 @@
 dc_pin = digitalio.DigitalInOut(board.D9)
 
 # Initialize display
-from adafruit_rgb_display import ili9341, color565
 display = ili9341.ILI9341(spi, cs=cs_pin, dc=dc_pin)
 # Fill with black!
 display.fill(color565(0, 0, 0))