Handle speed not being in delay format for newer 0x35 devices (#438)

Author: bdraco

flux_led/base_device.py

@@ -1063,7 +1063,7 @@ def getCCT(self) -> tuple[int, int]:
     @property
     def speed(self) -> int:
         assert self.raw_state is not None
-        if self.protocol in ADDRESSABLE_PROTOCOLS:
+        if self._protocol is not None and not self._protocol.speed_is_delay:
             return self.raw_state.speed
         if self.protocol in CHRISTMAS_EFFECTS_PROTOCOLS:
             return utils.delayToSpeed(self.raw_state.green)

flux_led/protocol.py

@@ -416,6 +416,11 @@ def __init__(self) -> None:
         self._counter = -1
         super().__init__()
 
+    @property
+    def speed_is_delay(self) -> bool:
+        """If True the speed is a delay."""
+        return True
+
     @property
     def requires_turn_on(self) -> bool:
         """If True the device must be turned on before setting level/patterns/modes."""
@@ -1408,6 +1413,11 @@ class ProtocolLEDENET25Byte(ProtocolLEDENET9Byte):
 
     level_write_modes = LevelWriteModeData(ALL=0x00, COLORS=0xA1, WHITES=0xB1)
 
+    @property
+    def speed_is_delay(self) -> bool:
+        """If True the speed is a delay."""
+        return False
+
     @property
     def name(self) -> str:
         """The name of the protocol."""
@@ -1499,6 +1509,11 @@ def timer_len(self) -> int:
         """Return a single timer len."""
         return 14
 
+    @property
+    def speed_is_delay(self) -> bool:
+        """If True the speed is a delay."""
+        return False
+
 
 class ProtocolLEDENETAddressableA1(ProtocolLEDENETAddressableBase):
     def construct_request_strip_setting(self) -> bytearray:
@@ -2386,6 +2401,11 @@ def construct_levels_change(
 
 
 class ProtocolLEDENETAddressableChristmas(ProtocolLEDENETAddressableBase):
+    @property
+    def speed_is_delay(self) -> bool:
+        """If True the speed is a delay in ms."""
+        return True
+
     def construct_state_query(self) -> bytearray:
         """The bytes to send for a query request."""
         return self.construct_wrapped_message(

tests/test_sync.py

@@ -873,6 +873,7 @@ def read_data(expected):
         self.assertEqual(light.protocol, PROTOCOL_LEDENET_25BYTE)
         self.assertEqual(light.model_num, 0x35)
         self.assertEqual(light.version_num, 10)
+        self.assertEqual(light.speed, 64)
 
         light.setRgb(255, 100, 50)
         self.assertEqual(mock_send.call_count, 2)