ramses-rf 0.51.8__py3-none-any.whl → 0.52.0__py3-none-any.whl

ramses_rf/gateway.py

@@ -129,7 +129,7 @@ class Gateway(Engine):
         self.devices: list[Device] = []
         self.device_by_id: dict[DeviceIdT, Device] = {}
 
-        self.msg_db: MessageIndex | None = None  # MessageIndex()
+        self.msg_db: MessageIndex | None = None
 
     def __repr__(self) -> str:
         if not self.ser_name:
@@ -173,6 +173,11 @@ class Gateway(Engine):
             **self._packet_log,
         )
 
+        # initialize SQLite index, set in _tx/Engine
+        if self._sqlite_index:  # TODO(eb): default to ON in Q4 2025
+            self.create_sqlite_message_index()  # if activated in ramses_cc > Engine
+
+        # temporarily turn on discovery, remember original state
         self.config.disable_discovery, disable_discovery = (
             True,
             self.config.disable_discovery,
@@ -184,6 +189,7 @@ class Gateway(Engine):
         if cached_packets:
             await self._restore_cached_packets(cached_packets)
 
+        # reset discovery to original state
         self.config.disable_discovery = disable_discovery
 
         if (
@@ -193,6 +199,9 @@ class Gateway(Engine):
         ):
             initiate_discovery(self.devices, self.systems)
 
+    def create_sqlite_message_index(self) -> None:
+        self.msg_db = MessageIndex()  # start the index
+
     async def stop(self) -> None:
         """Stop the Gateway and tidy up."""
 
@@ -248,12 +257,13 @@ class Gateway(Engine):
             #     return True
             return include_expired or not msg._expired
 
-        msgs = [m for device in self.devices for m in device._msg_db]
+        msgs = [m for device in self.devices for m in device._msg_list]
 
         for system in self.systems:
             msgs.extend(list(system._msgs.values()))
             msgs.extend([m for z in system.zones for m in z._msgs.values()])
-            # msgs.extend([m for z in system.dhw for m in z._msgs.values()])  # TODO
+            # msgs.extend([m for z in system.dhw for m in z._msgs.values()])  # TODO: DHW
+            # Related to/Fixes ramses_cc Issue 249 non-existing via-device _HW ?
 
         if self.msg_db:
             pkts = {
@@ -261,8 +271,8 @@ class Gateway(Engine):
                 for msg in self.msg_db.all(include_expired=True)
                 if wanted_msg(msg, include_expired=include_expired)
             }
-
-        else:
+        else:  # deprecated, to be removed in Q1 2026
+            # _LOGGER.warning("Missing MessageIndex")
             pkts = {  # BUG: assumes pkts have unique dtms: may be untrue for contrived logs
                 f"{repr(msg._pkt)[:26]}": f"{repr(msg._pkt)[27:]}"
                 for msg in msgs
@@ -359,7 +369,7 @@ class Gateway(Engine):
         """
 
         def check_filter_lists(dev_id: DeviceIdT) -> None:  # may: LookupError
-            """Raise an LookupError if a device_id is filtered out by a list."""
+            """Raise a LookupError if a device_id is filtered out by a list."""
 
             if dev_id in self._unwanted:  # TODO: shouldn't invalidate a msg
                 raise LookupError(f"Can't create {dev_id}: it is unwanted or invalid")

ramses_rf/version.py

@@ -1,4 +1,4 @@
 """RAMSES RF - a RAMSES-II protocol decoder & analyser (application layer)."""
 
-__version__ = "0.51.8"
+__version__ = "0.52.0"
 VERSION = __version__

{ramses_rf-0.51.8.dist-info → ramses_rf-0.52.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ramses_rf
-Version: 0.51.8
+Version: 0.52.0
 Summary: A stateful RAMSES-II protocol decoder & analyser.
 Project-URL: Homepage, https://github.com/ramses-rf/ramses_rf
 Project-URL: Bug Tracker, https://github.com/ramses-rf/ramses_rf/issues

{ramses_rf-0.51.8.dist-info → ramses_rf-0.52.0.dist-info}/RECORD

@@ -1,55 +1,55 @@
 ramses_cli/__init__.py,sha256=uvGzWqOf4avvgzxJNSLFWEelIWqSZ-AeLAZzg5x58bc,397
-ramses_cli/client.py,sha256=vbKS3KVPiGsDWLp5cR3SVBtXrs-TinzlxSbTgcb4G2k,19724
+ramses_cli/client.py,sha256=NTLhHhTiYmp7nyE3vqIFR9zjwfot1wdIT7QuMUgubD0,20350
 ramses_cli/debug.py,sha256=vgR0lOHoYjWarN948dI617WZZGNuqHbeq6Tc16Da7b4,608
-ramses_cli/discovery.py,sha256=81XbmpNiCpUHVZBwo2g1eRwyJG-wZhpSsc44G3hHlFA,12972
+ramses_cli/discovery.py,sha256=MWoahBnAAVzfK2S7EDLsY2WYqN_ZK9L-lktrj8_4cb0,12978
 ramses_cli/utils/cat_slow.py,sha256=AhUpM5gnegCitNKU-JGHn-DrRzSi-49ZR1Qw6lxe_t8,607
 ramses_cli/utils/convert.py,sha256=D_YiCyX5na9pgC-_NhBlW9N1dgRKUK-uLtLBfofjzZM,1804
 ramses_rf/__init__.py,sha256=VG3E9GHbtC6lx6E1DMQJeFitHnydMKJyPxQBethdrzg,1193
 ramses_rf/binding_fsm.py,sha256=uZAOl3i19KCXqqlaLJWkEqMMP7NJBhVPW3xTikQD1fY,25996
 ramses_rf/const.py,sha256=L3z31CZ-xqno6oZp_h-67CB_5tDDqTwSWXsqRtsjMcs,5460
-ramses_rf/database.py,sha256=ZZZgucyuU1IHsSewGukZfDg2gu8KeNaEFriWKM0TUHs,10287
-ramses_rf/dispatcher.py,sha256=JGkqSi1o-YhQ2rj8tNkXwYLLeJIC7F061xpHoH8sSsM,11201
-ramses_rf/entity_base.py,sha256=Byt8mFRUKETNiKHaL0cNaMywjLcopDG3Sldiy1Q7lAo,39213
+ramses_rf/database.py,sha256=21M1XV7qTQaydqInctf7vX-PAVdrM4xmJoaXX8oB87s,20748
+ramses_rf/dispatcher.py,sha256=UqilAoL6T85ZFPkcpvcfuidNbusONngQoCVWHV6vOzk,11277
+ramses_rf/entity_base.py,sha256=eLGhJTv8FDHjXmsywUhn1imEhdo0bFgAHP_KLNl_HiE,56004
 ramses_rf/exceptions.py,sha256=mt_T7irqHSDKir6KLaf6oDglUIdrw0S40JbOrWJk5jc,3657
-ramses_rf/gateway.py,sha256=s2bhkUzR42mzL4lZ1crTBsEWMOMGI8tpuHN1UZdAB74,20564
+ramses_rf/gateway.py,sha256=1MRmqpDlhE1SFm25dSvnSTX_dUoi9Pz--qycJIJyFAU,21151
 ramses_rf/helpers.py,sha256=TNk_QkpIOB3alOp1sqnA9LOzi4fuDCeapNlW3zTzNas,4250
 ramses_rf/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ramses_rf/schemas.py,sha256=UhvRhV4nZ3kvrLLM3wwIguQUIjIgd_AKvp2wkTSpNEA,13468
-ramses_rf/version.py,sha256=PUTZH-D81qyH8onRDdeZzxGwuCVDzMYTbItYAp9NKKI,125
+ramses_rf/version.py,sha256=ggzTkhC_7XHqI8ONDXEhJk9tXE1LLvMfYJyCrJzF8KI,125
 ramses_rf/device/__init__.py,sha256=sUbH5dhbYFXSoM_TPFRutpRutBRpup7_cQ9smPtDTy8,4858
-ramses_rf/device/base.py,sha256=WGkBTUNjRUEe-phxdtdiXVCZnTi6-i1i_YT6g689UTM,17450
-ramses_rf/device/heat.py,sha256=2sCsggySVcuTzyXDmgWy76QbhlU5MQWSejy3zgI5BDE,54242
-ramses_rf/device/hvac.py,sha256=uKljqweushvpHwRTiYfR26YST6--KAIp8X9zyiRv9HI,45607
+ramses_rf/device/base.py,sha256=Yx0LZwMEb49naY8FolZ8HEBFb6XCPQBTVN_TWyO2nKg,17777
+ramses_rf/device/heat.py,sha256=UmZuA-5czrfoClVEeUAPWgze5obWNQpYI7ZPQpVJB6s,54704
+ramses_rf/device/hvac.py,sha256=of8wWHhJOX0KcvVqqlyJLArKrv1ST7rlbL7kLV9v_0Q,45603
 ramses_rf/system/__init__.py,sha256=uZLKio3gLlBzePa2aDQ1nxkcp1YXOGrn6iHTG8LiNIw,711
 ramses_rf/system/faultlog.py,sha256=GdGmVGT3137KsTlV_nhccgIFEmYu6DFsLTn4S-8JSok,12799
 ramses_rf/system/heat.py,sha256=3jaFEChU-HlWCRMY1y7u09s7AH4hT0pC63hnqwdmZOc,39223
 ramses_rf/system/schedule.py,sha256=Ts6tdZPTQLV5NkgwA73tPa5QUsnZNIIuYoKC-8VsXDk,18808
 ramses_rf/system/zones.py,sha256=9AH7ooN5QfiqvWuor2P1Dn8aILjQb2RWL9rWqDH1IjA,36075
 ramses_tx/__init__.py,sha256=qNMTe8hBkIuecvtCiekUB0pKdD8atb0SjWxVNVe3yhE,3538
-ramses_tx/address.py,sha256=5swDr_SvOs1CxBmT-iJpldf8R00mOb7gKPMiEnxLz84,8452
+ramses_tx/address.py,sha256=F5ZE-EbPNNom1fW9XXUILvD7DYSMBxNJvsHVliT5gjw,8452
 ramses_tx/command.py,sha256=r9dNaofjjOQXZSUrZjsNpvEukNn4rSGy0OLr2Dyd2TI,125129
-ramses_tx/const.py,sha256=QmwSS4BIN3ZFrLUiiFScP1RCUHuJ782V3ycRPQTtB_c,30297
+ramses_tx/const.py,sha256=pnxq5upXvLUizv9Ye_I1llD9rAa3wddHgsSkc91AIUc,30300
 ramses_tx/exceptions.py,sha256=FJSU9YkvpKjs3yeTqUJX1o3TPFSe_B01gRGIh9b3PNc,2632
 ramses_tx/fingerprints.py,sha256=nfftA1E62HQnb-eLt2EqjEi_la0DAoT0wt-PtTMie0s,11974
-ramses_tx/frame.py,sha256=9lUVh8gAMXNRAolfFw2WuWANjn24AWkmscuM9Tm5imE,22036
-ramses_tx/gateway.py,sha256=TXLYwT6tFpmSokD29Qyj1ze7UGCxKidooeyP557Jfoo,11266
+ramses_tx/frame.py,sha256=GzNsXr15YLeidJYGtk_xPqsZQh4ehDDlUCtT6rTDhT8,22046
+ramses_tx/gateway.py,sha256=ztPg3fMgn-fM15dLxrw5PG34SCxD7eR3pdz1xwvz7Ag,11345
 ramses_tx/helpers.py,sha256=J4OCRckp3JshGQTvvqEskFjB1hPS7uA_opVsuIqmZds,32915
 ramses_tx/logger.py,sha256=qYbUoNPnPaFWKVsYvLG6uTVuPTdZ8HsMzBbGx0DpBqc,10177
-ramses_tx/message.py,sha256=hl_gLfwrF79ftUNnsgNt3XGsIhM2Pts0MtZZuGjfaxk,13169
+ramses_tx/message.py,sha256=-moQ8v3HVlNSl-x3U0DDfDcj8WQ7vLqclMNxsohbmnw,13449
 ramses_tx/opentherm.py,sha256=58PXz9l5x8Ou6Fm3y-R_UnGHCYahoi2RKIDdYStUMzk,42378
-ramses_tx/packet.py,sha256=NGunaGCkEjhTp9t4mARK5e7kbqT-Z_JKCH7ibMYMJXU,7357
-ramses_tx/parsers.py,sha256=c6D3NB79vZfOGpVmrHc-EhSh4_N4tKljBp1J3HFfkXw,109950
+ramses_tx/packet.py,sha256=_qHiPFWpQpKueZOgf1jJ93Y09iZjo3LZWStLglVkXg4,7370
+ramses_tx/parsers.py,sha256=e6IwVEMLv2EL8gbZaM2s-qA3E2AN8dCwdIgfEbFo730,111029
 ramses_tx/protocol.py,sha256=9R3aCzuiWEyXmugmB5tyR_RhBlgfnpUXj7AsMP9BzzU,28867
 ramses_tx/protocol_fsm.py,sha256=ZKtehCr_4TaDdfdlfidFLJaOVTYtaEq5h4tLqNIhb9s,26827
 ramses_tx/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ramses_tx/ramses.py,sha256=DujcZe4WelHvPNKYfoz9YWmXNoCayV5UsAkiu8Y6vms,53432
-ramses_tx/schemas.py,sha256=18IPRdoCWXpcRg4v8Z1ehTnRronQPYGrf4AvRL-1OD0,12932
-ramses_tx/transport.py,sha256=MwPnkQ0L-2qJt4mIJy3-C9XmHwBDjT7Kg-1LthPByVw,58331
+ramses_tx/ramses.py,sha256=NG81GBNZlap-Gi9ac-r6OFE-KaHvXsgPDWy-I2Irr-4,53698
+ramses_tx/schemas.py,sha256=IYCDH0jp3446Gtl_378aBmWyXN90e-uYudfkZkOKR24,13147
+ramses_tx/transport.py,sha256=bGprlfuuwBgQ1bmBRSrcicuk7s-jVqyuKpZCfQ-sSpw,58469
 ramses_tx/typed_dicts.py,sha256=w-0V5t2Q3GiNUOrRAWiW9GtSwbta_7luME6GfIb1zhI,10869
 ramses_tx/typing.py,sha256=eF2SlPWhNhEFQj6WX2AhTXiyRQVXYnFutiepllYl2rI,5042
-ramses_tx/version.py,sha256=kLXw7G-2GHtLEZnlOm_5jj8TGeibQYmH_NZ9F7FVHAU,123
-ramses_rf-0.51.8.dist-info/METADATA,sha256=JmxHDvvXI3gRkt2zHELxS9G3ZaMyOn8bOvaraq5_0Ro,4000
-ramses_rf-0.51.8.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-ramses_rf-0.51.8.dist-info/entry_points.txt,sha256=NnyK29baOCNg8DinPYiZ368h7MTH7bgTW26z2A1NeIE,50
-ramses_rf-0.51.8.dist-info/licenses/LICENSE,sha256=-Kc35W7l1UkdiQ4314_yVWv7vDDrg7IrJfMLUiq6Nfs,1074
-ramses_rf-0.51.8.dist-info/RECORD,,
+ramses_tx/version.py,sha256=8xCh-Ia7WXZkWWEs7lVuk13x3NbC3HoNuHvR_G1KCB4,123
+ramses_rf-0.52.0.dist-info/METADATA,sha256=ADZlDrTcgK0-xOW4D1A-2CRfakYnaGOOClGRshQYBwg,4000
+ramses_rf-0.52.0.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+ramses_rf-0.52.0.dist-info/entry_points.txt,sha256=NnyK29baOCNg8DinPYiZ368h7MTH7bgTW26z2A1NeIE,50
+ramses_rf-0.52.0.dist-info/licenses/LICENSE,sha256=-Kc35W7l1UkdiQ4314_yVWv7vDDrg7IrJfMLUiq6Nfs,1074
+ramses_rf-0.52.0.dist-info/RECORD,,

ramses_tx/address.py

@@ -195,7 +195,7 @@ def pkt_addrs(addr_fragment: str) -> tuple[Address, Address, Address, Address, A
 
     returns: src_addr, dst_addr, addr_0, addr_1, addr_2
 
-    Will raise an InvalidAddrSetError is the address fields are not valid.
+    Will raise an InvalidAddrSetError if the address fields are not valid.
     """
     # for debug: print(pkt_addrs.cache_info())
 

ramses_tx/const.py

@@ -788,7 +788,7 @@ class MsgId(StrEnum):
     _7F = "7F"
 
 
-# StrEnum is intended include all known codes, see: test suite, code schema in ramses.py
+# StrEnum is intended to include all known codes, see: test suite, code schema in ramses.py
 @verify(EnumCheck.UNIQUE)
 class Code(StrEnum):
     _0001 = "0001"

ramses_tx/frame.py

@@ -65,7 +65,7 @@ class Frame:
     def __init__(self, frame: str) -> None:
         """Create a frame from a string.
 
-        Will raise InvalidPacketError if it is invalid.
+        :raises InvalidPacketError: if provided string is invalid.
         """
 
         self._frame: str = frame
@@ -123,7 +123,7 @@ class Frame:
         if not strict_checking:
             return
 
-        try:  # Strict checking: helps users avoid to constructing bad commands
+        try:  # Strict checking: helps users avoid constructing bad commands
             if addrs[0] == NON_DEV_ADDR:
                 assert self.verb == I_, "wrong verb or dst addr should be present"
             elif addrs[2] == NON_DEV_ADDR:
@@ -138,7 +138,7 @@ class Frame:
             raise exc.PacketInvalid(f"Bad frame: Invalid address set: {err}") from err
 
     def __repr__(self) -> str:
-        """Return a unambiguous string representation of this object."""
+        """Return an unambiguous string representation of this object."""
 
         if self._repr is None:
             self._repr = " ".join(  # type: ignore[unreachable]
@@ -387,7 +387,7 @@ class Frame:
 
     @property
     def _hdr(self) -> HeaderT:  # incl. self._ctx
-        """Return the QoS header (fingerprint) of this packet (i.e. device_id/code/hdr).
+        """Return the QoS header (fingerprint) of this packet (i.e. device_id|code|verb).
 
         Used for QoS (timeouts, retries), callbacks, etc.
         """

ramses_tx/gateway.py

@@ -37,6 +37,7 @@ from .schemas import (
     SZ_PACKET_LOG,
     SZ_PORT_CONFIG,
     SZ_PORT_NAME,
+    SZ_SQLITE_INDEX,
     PktLogConfigT,
     PortConfigT,
     select_device_filter_mode,
@@ -91,7 +92,7 @@ class Engine:
         if input_file:
             self._disable_sending = True
         elif not port_name:
-            raise TypeError("Either a port_name or a input_file must be specified")
+            raise TypeError("Either a port_name or an input_file must be specified")
 
         self.ser_name = port_name
         self._input_file = input_file
@@ -112,6 +113,7 @@ class Engine:
             self._include,
             self._exclude,
         )
+        self._sqlite_index = kwargs.pop(SZ_SQLITE_INDEX)
         self._kwargs: dict[str, Any] = kwargs  # HACK
 
         self._engine_lock = Lock()  # FIXME: threading lock, or asyncio lock?

ramses_tx/message.py

@@ -54,7 +54,7 @@ class MessageBase:
     def __init__(self, pkt: Packet) -> None:
         """Create a message from a valid packet.
 
-        Will raise InvalidPacketError if it is invalid.
+        :raises InvalidPacketError if message payload is invalid.
         """
 
         self._pkt = pkt
@@ -66,11 +66,15 @@ class MessageBase:
         self.dtm: dt = pkt.dtm
 
         self.verb: VerbT = pkt.verb
-        self.seqn: str = pkt.seqn
+        self.seqn: str = (
+            pkt.seqn
+        )  # the msg is part of a set for 1 Code, received in order
         self.code: Code = pkt.code
         self.len: int = pkt._len
 
-        self._payload = self._validate(self._pkt.payload)  # ? raise InvalidPacketError
+        self._payload = self._validate(
+            self._pkt.payload
+        )  # ? may raise InvalidPacketError
 
         self._str: str = None  # type: ignore[assignment]
 
@@ -92,7 +96,9 @@ class MessageBase:
 
         if self.src.id == self._addrs[0].id:  # type: ignore[unreachable]
             name_0 = self._name(self.src)
-            name_1 = "" if self.dst is self.src else self._name(self.dst)
+            name_1 = (
+                "" if self.dst is self.src else self._name(self.dst)
+            )  # use 'is', issue_cc 318
         else:
             name_0 = ""
             name_1 = self._name(self.src)
@@ -139,16 +145,18 @@ class MessageBase:
 
     @property
     def _has_array(self) -> bool:
-        """Return True if the message's raw payload is an array."""
+        """
+        :return: True if the message's raw payload is an array.
+        """
 
         return bool(self._pkt._has_array)
 
     @property
     def _idx(self) -> dict[str, str]:
-        """Return the domain_id/zone_idx/other_idx of a message payload, if any.
+        """Get the domain_id/zone_idx/other_idx of a message payload, if any.
+        Used to identify the zone/domain that a message applies to.
 
-        Used to identify the zone/domain that a message applies to. Returns an empty
-        dict if there is none such, or None if undetermined.
+        :return: an empty dict if there is none such, or None if undetermined.
         """
 
         # .I --- 01:145038 --:------ 01:145038 3B00 002 FCC8
@@ -229,7 +237,7 @@ class MessageBase:
             assert isinstance(self._pkt._idx, str)  # mypy hint
             return {IDX_NAMES[Code._22C9]: self._pkt._idx}
 
-        assert isinstance(self._pkt._idx, str)  # mypy check
+        assert isinstance(self._pkt._idx, str)  # mypy hint
         idx_name = SZ_DOMAIN_ID if self._pkt._idx[:1] == "F" else SZ_ZONE_IDX
         index_name = IDX_NAMES.get(self.code, idx_name)
 
@@ -237,9 +245,10 @@ class MessageBase:
 
     # TODO: needs work...
     def _validate(self, raw_payload: str) -> dict | list[dict]:  # type: ignore[type-arg]
-        """Validate the message, and parse the payload if so.
+        """Validate a message packet payload, and parse it if valid.
 
-        Raise an exception (InvalidPacketError) if it is not valid.
+        :return: a dict containing key: value pairs, or a list of those created from the payload
+        :raises an InvalidPacketError exception if it is not valid.
         """
 
         try:  # parse the payload
@@ -249,10 +258,9 @@ class MessageBase:
             if not self._has_payload and (
                 self.verb == RQ and self.code not in RQ_IDX_COMPLEX
             ):
-                # _LOGGER.error("%s", msg)
                 return {}
 
-            result = parse_payload(self)
+            result = parse_payload(self)  # invoke the code parsers
 
             if isinstance(result, list):
                 return result
@@ -353,7 +361,7 @@ def re_compile_re_match(regex: str, string: str) -> bool:  # Optional[Match[Any]
 def _check_msg_payload(msg: MessageBase, payload: str) -> None:
     """Validate the packet's payload against its verb/code pair.
 
-    Raise an InvalidPayloadError if the payload is seen as invalid. Such payloads may
+    :raises InvalidPayloadError if the payload is seen as invalid. Such payloads may
     actually be valid, in which case the rules (likely the regex) will need updating.
     """
 

ramses_tx/packet.py

@@ -104,7 +104,7 @@ class Packet(Frame):
         return f"{dtm} ... {self}{hdr}"
 
     def __str__(self) -> str:
-        """Return a brief readable string representation of this object."""
+        """Return a brief readable string representation of this object aka 'header'."""
         # e.g.: 000A|RQ|01:145038|08
         return super().__repr__()  # TODO: self._hdr
 

ramses_tx/parsers.py

@@ -1912,42 +1912,59 @@ def parser_2411(payload: str, msg: Message) -> dict[str, Any]:
         "92": (4, hex_to_temp),  # 75 (0-30) (C)
     }  # TODO: _2411_TYPES.get(payload[8:10], (8, no_op))
 
-    assert payload[4:6] in _2411_TABLE, (
-        f"param {payload[4:6]} is unknown"
-    )  # _INFORM_DEV_MSG
-    description = _2411_TABLE.get(payload[4:6], "Unknown")
+    # Handle unknown parameters gracefully instead of asserting
+    param_id = payload[4:6]
+    try:
+        description = _2411_TABLE.get(param_id, "Unknown")
+        if param_id not in _2411_TABLE:
+            _LOGGER.warning(
+                f"2411 message received with unknown parameter ID: {param_id}. "
+                f"This parameter is not in the known parameter schema. "
+                f"Message: {msg!r}"
+            )
+    except Exception as err:
+        _LOGGER.warning(f"Error looking up 2411 parameter {param_id}: {err}")
+        description = "Unknown"
 
     result = {
-        "parameter": payload[4:6],
+        "parameter": param_id,
         "description": description,
     }
 
     if msg.verb == RQ:
         return result
 
-    assert payload[8:10] in _2411_DATA_TYPES, (
-        f"param {payload[4:6]} has unknown data_type: {payload[8:10]}"
-    )  # _INFORM_DEV_MSG
-    length, parser = _2411_DATA_TYPES.get(payload[8:10], (8, lambda x: x))
-
-    result |= {
-        "value": parser(payload[10:18][-length:]),  # type: ignore[operator]
-        "_value_06": payload[6:10],
-    }
+    try:
+        assert payload[8:10] in _2411_DATA_TYPES, (
+            f"param {param_id} has unknown data_type: {payload[8:10]}"
+        )  # _INFORM_DEV_MSG
+        length, parser = _2411_DATA_TYPES.get(payload[8:10], (8, lambda x: x))
+
+        result |= {
+            "value": parser(payload[10:18][-length:]),  # type: ignore[operator]
+            "_value_06": payload[6:10],
+        }
 
-    if msg.len == 9:
+        if msg.len == 9:
+            return result
+
+        return (
+            result
+            | {
+                "min_value": parser(payload[18:26][-length:]),  # type: ignore[operator]
+                "max_value": parser(payload[26:34][-length:]),  # type: ignore[operator]
+                "precision": parser(payload[34:42][-length:]),  # type: ignore[operator]
+                "_value_42": payload[42:],
+                # Flexible footer - capture everything after precision
+                # eg. older Orcon models may have a footer of 2 bytes
+            }
+        )
+    except AssertionError as err:
+        _LOGGER.warning(f"{msg!r} < {_INFORM_DEV_MSG} ({err})")
+        # Return partial result for unknown parameters
+        result["value"] = ""
         return result
 
-    return (
-        result
-        | {
-            "min_value": parser(payload[18:26][-length:]),  # type: ignore[operator]
-            "max_value": parser(payload[26:34][-length:]),  # type: ignore[operator]
-            "precision": parser(payload[34:42][-length:]),  # type: ignore[operator]
-            "_value_42": payload[42:],
-        }
-    )
-
 
 # unknown_2420, from OTB
 def parser_2420(payload: str, msg: Message) -> dict[str, Any]:
@@ -2347,7 +2364,7 @@ def parser_3210(payload: str, msg: Message) -> PayDictT._3210:
     return {SZ_TEMPERATURE: hex_to_temp(payload[2:])}
 
 
-# opentherm_msg, from OTB (and some RND)
+# opentherm_msg, from OTB (and OT_RND)
 def parser_3220(payload: str, msg: Message) -> dict[str, Any]:
     try:
         ot_type, ot_id, ot_value, ot_schema = decode_frame(payload[2:10])
@@ -2971,8 +2988,12 @@ _PAYLOAD_PARSERS = {
 
 
 def parse_payload(msg: Message) -> dict | list[dict]:
+    """
+    Apply the appropriate parser defined in this module to the message.
+    :param msg: a Message object containing packet data and extra attributes
+    :return: a dict of key: value pairs or a list of such dicts, e.g. {'temperature': 21.5}
+    """
     result: dict | list[dict]
-
     result = _PAYLOAD_PARSERS.get(msg.code, parser_unknown)(msg._pkt.payload, msg)
     if isinstance(result, dict) and msg.seqn.isnumeric():  # e.g. 22F1/3
         result["seqx_num"] = msg.seqn

ramses_tx/ramses.py

@@ -478,10 +478,10 @@ CODES_SCHEMA: dict[Code, dict[str, Any]] = {  # rf_unknown
     },
     Code._2411: {  # fan_params, HVAC
         SZ_NAME: "fan_params",
-        I_: r"^(00|01|15|16|17|21)00[0-9A-F]{6}([0-9A-F]{8}){4}[0-9A-F]{4}$",
+        I_: r"^(00|01|15|16|17|21)00[0-9A-F]{6}([0-9A-F]{8}){4}[0-9A-F]{2,6}$",  # Allow 2-6 byte footer
         RQ: r"^(00|01|15|16|17|21)00[0-9A-F]{2}((00){19})?$",
-        RP: r"^(00|01|15|16|17|21)00[0-9A-F]{6}[0-9A-F]{8}(([0-9A-F]{8}){3}[0-9A-F]{4})?$",
-        W_: r"^(00|01|15|16|17|21)00[0-9A-F]{6}[0-9A-F]{8}(([0-9A-F]{8}){3}[0-9A-F]{4})?$",
+        RP: r"^(00|01|15|16|17|21)00[0-9A-F]{6}[0-9A-F]{8}([0-9A-F]{8}){3}[0-9A-F]{2,6}$",  # 4 blocks + footer
+        W_: r"^(00|01|15|16|17|21)00[0-9A-F]{6}[0-9A-F]{8}([0-9A-F]{8}){3}[0-9A-F]{2,6}$",  # Same as RP
     },
     Code._2420: {  # unknown_2420, from OTB
         SZ_NAME: "message_2420",
@@ -1267,6 +1267,14 @@ _22F1_SCHEMES: dict[str, dict[str, str]] = {
 
 # unclear if true for only Orcon/*all* models
 _2411_PARAMS_SCHEMA: dict[str, dict[str, Any]] = {
+    "01": {  # all?
+        SZ_DESCRIPTION: "Support",
+        SZ_MIN_VALUE: 0xFF,  # None?
+        SZ_MAX_VALUE: 0xFF,
+        SZ_PRECISION: 1,
+        SZ_DATA_TYPE: "00",
+        SZ_DATA_UNIT: "",
+    },
     "31": {  # slot 09 (FANs produced after 2021)
         SZ_DESCRIPTION: "Time to change filter (days)",
         SZ_MIN_VALUE: 0,

ramses_tx/schemas.py

@@ -338,7 +338,6 @@ SCH_GLOBAL_TRAITS_DICT, SCH_TRAITS = sch_global_traits_dict_factory()
 #
 # Device lists (Engine configuration)
 
-
 DeviceIdT = NewType("DeviceIdT", str)  # TypeVar('DeviceIdT', bound=str)  #
 DevIndexT = NewType("DevIndexT", str)
 DeviceListT: TypeAlias = dict[DeviceIdT, DeviceTraitsT]
@@ -397,10 +396,14 @@ def select_device_filter_mode(
 
 #
 # 5/5: Gateway (engine) configuration
+
 SZ_DISABLE_SENDING: Final = "disable_sending"
 SZ_DISABLE_QOS: Final = "disable_qos"
 SZ_ENFORCE_KNOWN_LIST: Final[str] = f"enforce_{SZ_KNOWN_LIST}"
 SZ_EVOFW_FLAG: Final = "evofw_flag"
+SZ_SQLITE_INDEX: Final = (
+    "sqlite_index"  # temporary 0.52.x SQLite dev config option in ramses_cc
+)
 SZ_USE_REGEX: Final = "use_regex"
 
 SCH_ENGINE_DICT = {
@@ -408,10 +411,13 @@ SCH_ENGINE_DICT = {
     vol.Optional(SZ_DISABLE_QOS, default=None): vol.Any(
         None,  # None is selective QoS (e.g. QoS only for bindings, schedule, etc.)
         bool,
-    ),  # in long term, this default to be True (and no None)
+    ),  # in the long term, this default to be True (not None)
     vol.Optional(SZ_ENFORCE_KNOWN_LIST, default=False): bool,
     vol.Optional(SZ_EVOFW_FLAG): vol.Any(None, str),
     # vol.Optional(SZ_PORT_CONFIG): SCH_SERIAL_PORT_CONFIG,
+    vol.Optional(
+        SZ_SQLITE_INDEX, default=False
+    ): bool,  # temporary 0.52.x dev config option
     vol.Optional(SZ_USE_REGEX): dict,  # vol.All(ConvertNullToDict(), dict),
     vol.Optional(SZ_COMMS_PARAMS): SCH_COMMS_PARAMS,
 }

ramses_tx/transport.py

@@ -44,7 +44,7 @@ import sys
 from collections import deque
 from collections.abc import Awaitable, Callable, Iterable
 from datetime import datetime as dt, timedelta as td
-from functools import wraps
+from functools import partial, wraps
 from io import TextIOWrapper
 from string import printable
 from time import perf_counter
@@ -142,8 +142,7 @@ else:  # is linux
 
     def list_links(devices: set[str]) -> list[str]:
         """Search for symlinks to ports already listed in devices."""
-
-        links = []
+        links: list[str] = []
         for device in glob.glob("/dev/*") + glob.glob("/dev/serial/by-id/*"):
             if os.path.islink(device) and os.path.realpath(device) in devices:
                 links.append(device)
@@ -174,7 +173,7 @@ else:  # is linux
         return result
 
 
-def is_hgi80(serial_port: SerPortNameT) -> bool | None:
+async def is_hgi80(serial_port: SerPortNameT) -> bool | None:
     """Return True/False if the device attached to the port has the attrs of an HGI80.
 
     Return None if it's not possible to tell (falsy should assume is evofw3).
@@ -209,7 +208,10 @@ def is_hgi80(serial_port: SerPortNameT) -> bool | None:
 
     # otherwise, we can look at device attrs via comports()...
     try:
-        komports = comports(include_links=True)
+        loop = asyncio.get_running_loop()
+        komports = await loop.run_in_executor(
+            None, partial(comports, include_links=True)
+        )
     except ImportError as err:
         raise exc.TransportSerialError(f"Unable to find {serial_port}: {err}") from err
 
@@ -841,8 +843,6 @@ class PortTransport(_RegHackMixin, _FullTransport, _PortTransportAbstractor):  #
             self._leak_sem(), name="PortTransport._leak_sem()"
         )
 
-        self._is_hgi80 = is_hgi80(self.serial.name)
-
         self._loop.create_task(
             self._create_connection(), name="PortTransport._create_connection()"
         )
@@ -855,6 +855,8 @@ class PortTransport(_RegHackMixin, _FullTransport, _PortTransportAbstractor):  #
 
         # signature also serves to discover the HGI's device_id (& for pkt log, if any)
 
+        self._is_hgi80 = await is_hgi80(self.serial.name)
+
         async def connect_sans_signature() -> None:
             """Call connection_made() without sending/waiting for a signature."""
 

ramses_tx/version.py

@@ -1,4 +1,4 @@
 """RAMSES RF - a RAMSES-II protocol decoder & analyser (transport layer)."""
 
-__version__ = "0.51.8"
+__version__ = "0.52.0"
 VERSION = __version__