Cosmetics

README.md

@@ -337,8 +337,8 @@ Then push green button on MoveHub, so permanent BLE connection will be establish
 
 ## Links
 
+- https://github.com/LEGO/lego-ble-wireless-protocol-docs - true docs for LEGO BLE protocol
 - https://github.com/JorgePe/BOOSTreveng - initial source of protocol knowledge
-- https://github.com/LEGO/lego-ble-wireless-protocol-docs - true docs of BLE protocol
 - https://github.com/spezifisch/sphero-python/blob/master/BB8joyDrive.py - example with another approach to bluetooth libs
 
 Some things around visual programming:

examples/automata/__init__.py

@@ -1,7 +1,7 @@
 import time
 from collections import Counter
 
-from pylgbst.constants import COLORS, COLOR_NONE, COLOR_BLACK, COLOR_CYAN, COLOR_BLUE
+from pylgbst.constants import COLOR_NONE, COLOR_BLACK, COLOR_CYAN, COLOR_BLUE
 from pylgbst.movehub import MoveHub
 from pylgbst.peripherals import ColorDistanceSensor
 
@@ -47,12 +47,3 @@ class Automata(object):
     def backward(self):
         self.__hub.motor_AB.angled(830, -0.25)
         time.sleep(0.5)
-
-
-automata = Automata()
-
-for x in range(16):
-    automata.feed_tape()
-    print (x, COLORS[automata.get_color()])
-
-# time.sleep(1000)

pylgbst/__init__.py

@@ -7,6 +7,7 @@ log = logging.getLogger('pylgbst')
 
 
 def get_connection_bluegiga(controller=None, hub_mac=None):
+    del controller  # to prevent code analysis warning
     from pylgbst.comms.cpygatt import BlueGigaConnection
 
     return BlueGigaConnection().connect(hub_mac)
@@ -29,6 +30,7 @@ def get_connection_gattlib(controller='hci0', hub_mac=None):
 
     return GattLibConnection(controller).connect(hub_mac)
 
+
 def get_connection_bluepy(controller='hci0', hub_mac=None):
     from pylgbst.comms.cbluepy import BluepyConnection
 

pylgbst/constants.py

@@ -38,6 +38,8 @@ MSG_DEVICE_SHUTDOWN = 0x02  # sent when hub shuts down by button hold
 MSG_PING_RESPONSE = 0x03
 MSG_PORT_INFO = 0x04
 MSG_PORT_CMD_ERROR = 0x05
+
+
 MSG_SET_PORT_VAL = 0x81
 MSG_PORT_STATUS = 0x82
 MSG_SENSOR_SUBSCRIBE = 0x41

pylgbst/movehub.py

@@ -244,3 +244,7 @@ class MoveHub(object):
             time.sleep(0.05)
 
         return self.info[info_type]
+
+    def __del__(self):
+        if self.connection and self.connection.is_alive():
+            self.connection.disconnect()