Merge branch 'ui_fix' into 'master'

Fix for issue #1 -- https://githepia.hesge.ch/adrienma.lescourt/actuasim_iot/issues/1

See merge request adrienma.lescourt/actuasim_iot!1

actuasim.py

@@ -150,7 +150,12 @@ class Actuasim(QMainWindow):
 
     def frame_received(self, frame):
         self.logger.info('Frame received:' + str([hex(h) for h in frame[0]]))
+        try:
             decoded_frame = knxnet.decode_frame(frame[0])
+        except Exception as e:
+            self.logger.info('Bad frame: {}'.format(e))
+            return
+
         if decoded_frame.header.service_type_descriptor == knxnet.ServiceTypeDescriptor.CONNECTION_REQUEST:
             self.logger.info('= Connection request:\n' + str(decoded_frame))
             conn_resp = knxnet.create_frame(knxnet.ServiceTypeDescriptor.CONNECTION_RESPONSE,

actuasim/blind.py

@@ -23,14 +23,14 @@ class BlindWidget(QWidget):
     def __init__(self, individual_address, group_address, blind_position=0, animation_speed_ms=1500):
         super(BlindWidget, self).__init__()
 
-        self.ui = Ui_Blind()
-        self.ui.setupUi(self)
-
         self.logger = logging.getLogger()
 
         self.individual_address = individual_address
         self.group_address = group_address
 
+        self.ui = Ui_Blind()
+        self.ui.setupUi(self)
+
         # Progressbar init
         self.is_moving = False
         self.animation_speed = animation_speed_ms

actuasim/valve.py

@@ -25,14 +25,18 @@ class ValveWidget(QWidget):
 
     def __init__(self, individual_address, group_address, valve_position=45, animation_speed_ms=1500):
         super(ValveWidget, self).__init__()
-        self.ui = Ui_Valve()
-        self.ui.setupUi(self)
 
         self.logger = logging.getLogger()
 
         self.individual_address = individual_address
         self.group_address = group_address
+
         self._position = 0
+
+        self.ui = Ui_Valve()
+        self.ui.setupUi(self)
+
+
         self.position = valve_position
         self.setFixedWidth(220)