Add support for DALI 24-bit event messages

IEC 62386 part 103 defines "event" messages, which can be sent by
control devices to indicate various inputs or changes. This commit
introduces support for encoding and decoding these messages, as well as
a handful of new sequences for making it easier to work with them.

Support has been added for IEC 62386 part 301 "push button" devices,
further device types should be simple enough to add in later.
Unsupported types will be safely ignored by the library.

.github/workflows/run-tests.yml

@@ -23,7 +23,7 @@ jobs:
           if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
       - name: Lint with flake8
         run: |
-          flake8 dali --exclude=driver --count --ignore=E741,F403,F405 --show-source --statistics --max-line-length=127
+          flake8 dali --exclude=driver --count --ignore=E741,F403,F405,W503 --show-source --statistics --max-line-length=127
           # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
           #flake8 dali --exclude=driver --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
       - name: Test with pytest

dali/address.py

@@ -101,8 +101,8 @@ def __str__(self):
         return "<broadcast (control gear)>"
 
 
-# Broadcast alias provided for legacy purposes, new code should avoid ambiguity by using
-# either GearBroadcast or DeviceBroadcast
+# Broadcast alias provided for legacy purposes, new code should avoid
+# ambiguity by using either GearBroadcast or DeviceBroadcast
 Broadcast = GearBroadcast
 
 
@@ -153,8 +153,9 @@ def __str__(self):
         return "<broadcast unaddressed (control gear)>"
 
 
-# BroadcastUnaddressed alias provided for legacy purposes, new code should avoid
-# ambiguity by using either GearBroadcastUnaddressed or DeviceBroadcastUnaddressed
+# BroadcastUnaddressed alias provided for legacy purposes, new code should
+# avoid ambiguity by using either GearBroadcastUnaddressed or
+# DeviceBroadcastUnaddressed
 BroadcastUnaddressed = GearBroadcastUnaddressed
 
 
@@ -213,8 +214,8 @@ def __str__(self):
         return f"<group (control gear) {self.group}>"
 
 
-# Group alias provided for legacy purposes, new code should avoid ambiguity by using
-# either GearGroup or DeviceGroup
+# Group alias provided for legacy purposes, new code should avoid ambiguity
+# by using either GearGroup or DeviceGroup
 Group = GearGroup
 
 
@@ -287,8 +288,8 @@ def __str__(self):
         return f"<address (control gear) {self.address}>"
 
 
-# Short alias provided for legacy purposes, new code should avoid ambiguity by using
-# either GearShort or DeviceShort
+# Short alias provided for legacy purposes, new code should avoid ambiguity
+# by using either GearShort or DeviceShort
 Short = GearShort
 
 
@@ -343,6 +344,11 @@ class Instance:
     def __init__(self):
         raise NotImplementedError
 
+    @property
+    def value(self):
+        if hasattr(self, "_value"):
+            return self._value
+
     def add_to_frame(self, f):
         raise NotImplementedError
 

dali/command.py

@@ -221,7 +221,7 @@ def _register_subclass(cls, subclass):
         subs.append(subclass)
 
     @classmethod
-    def from_frame(cls, f, devicetype=0):
+    def from_frame(cls, f, devicetype=0, dev_inst_map=None):
         """Return a Command instance corresponding to the supplied frame.
 
         If the device type the command is intended for is known
@@ -230,6 +230,8 @@ def from_frame(cls, f, devicetype=0):
 
         :parameter frame: a forward frame
         :parameter devicetype: type of device frame is intended for
+        :param dev_inst_map: An instance of DeviceInstanceTypeMapper to
+        assist with DALI events
 
         :returns: Return a Command instance corresponding to the
         frame.  Returns None if there is no match.
@@ -241,7 +243,7 @@ def from_frame(cls, f, devicetype=0):
         subs = cls._framesizes.get(len(f), [])
 
         for c in subs:
-            r = c.from_frame(f, devicetype=devicetype)
+            r = c.from_frame(f, devicetype=devicetype, dev_inst_map=dev_inst_map)
             if r:
                 return r
 

dali/device/__init__.py

@@ -4,3 +4,6 @@
 part 103 ("General requirements - Control devices") and parts 3xx
 (particular requirements for control devices).
 """
+import dali.device.general
+import dali.device.sequences
+import dali.device.pushbutton  # noqa: F401