Merge pull request #27 from uduncanu/lywsd02-history

Change _process_history_data to use a single unpack
h4 · Jan 3, 2021 · 2ba1168 · 2ba1168
2 parents 2de71fc + c7a6da1
commit 2ba1168
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/lywsd02/client.py b/lywsd02/client.py
@@ -166,11 +166,7 @@ def _process_sensor_data(self, data):
         self._data = SensorData(temperature=temperature, humidity=humidity)
 
     def _process_history_data(self, data):
-
-        # TODO unpacking with IIhBhB in one step doesn't work
-        (idx, ts) = struct.unpack_from('II', data[0:8])
-        (max_temp, max_hum) = struct.unpack_from('hB', data[8:11])
-        (min_temp, min_hum) = struct.unpack_from('hB', data[11:14])
+        (idx, ts, max_temp, max_hum, min_temp, min_hum) = struct.unpack_from('<IIhBhB', data)
 
         ts = datetime.fromtimestamp(ts)
         min_temp /= 100