python-mobius 0.1.1__py3-none-any.whl → 0.1.2__py3-none-any.whl

mobius/__init__.py

@@ -41,7 +41,7 @@ from .discovery import (
     scan_for_mobius_devices, scan_for_mobius_devices_with_info, group_by_pan_id,
 )
 
-__version__ = "0.1.1"
+__version__ = "0.1.2"
 
 __all__ = [
     "__version__",

mobius/device.py

@@ -131,8 +131,29 @@ class MobiusDevice:
                     if attempt == 2:
                         raise
                     await asyncio.sleep(2.0)
-        await self._client.start_notify(CHAR_RX_DATA, self._on_rx_data)
-        await self._client.start_notify(CHAR_RX_FINAL, self._on_rx_final)
+
+        # IMPORTANT: from here on, self._client holds a real, already-open
+        # BLE connection. If anything below fails, connect() (== __aenter__)
+        # raises -- and per the async context manager protocol, a raising
+        # __aenter__ means __aexit__/disconnect() is NEVER called by the
+        # caller's `async with` block. Without this try/except, a failure
+        # here would leak an orphaned open connection on the adapter every
+        # single time it happens -- and since coordinators retry on a timer,
+        # that leak compounds over repeated failures until the adapter's
+        # real connection-slot pool is exhausted, independent of range or
+        # any other integration. Confirmed as the likely cause of "no
+        # available connection slot" errors appearing over time rather than
+        # immediately, reported against real hardware.
+        try:
+            await self._client.start_notify(CHAR_RX_DATA, self._on_rx_data)
+            await self._client.start_notify(CHAR_RX_FINAL, self._on_rx_final)
+        except Exception:
+            try:
+                if self._client.is_connected:
+                    await self._client.disconnect()
+            except Exception:
+                pass  # best-effort cleanup; the original error is what matters
+            raise
 
     async def disconnect(self):
         if self._client and self._client.is_connected:
@@ -345,12 +366,24 @@ class MobiusDevice:
 
     async def get_pump_telemetry(self) -> dict:
         """
-        Returns {"speed": int, "gph": int}. Both are genuinely read live from
-        the device -- confirmed via FlowRange.java, which is what the app's
-        dashboard actually queries for the pump gauge widget.
+        Returns {"speed": int, "speed_percent": float, "gph": int}. speed
+        and gph are genuinely read live from the device -- confirmed via
+        FlowRange.java, which is what the app's dashboard actually queries
+        for the pump gauge widget.
 
         "gph" comes from PhysicalValues/GallonsPerHour (int32 LE).
-        "speed" comes from MotorSpeed directly (int16).
+
+        "speed" comes from MotorSpeed directly (int16). CONFIRMED (not
+        inferred) to be a percentage of max pump power in tenths of a
+        percent, not RPM: the app's own display code does
+        `String.format("%.0f%%", Math.abs(currentSpeed / 10.0f))` on this
+        exact value (FlowFragment.java/WidgetViewCell.java). "speed_percent"
+        is that same abs(speed)/10 conversion, provided as a convenience --
+        the sign of the raw "speed" value encodes rotation direction
+        (reverse), matching the same convention as the schedule's MaxSpeed
+        parameter (see PumpPrimitiveValue), so check the sign of "speed"
+        yourself if direction matters; speed_percent is always non-negative.
+
         MotorRPM is defined in the protocol but never queried anywhere in the
         app -- not included here since it's unconfirmed whether firmware
         actually populates it.
@@ -361,7 +394,8 @@ class MobiusDevice:
         )
         speed = struct.unpack("<h", speed_raw[0])[0] if speed_raw else None
         gph = struct.unpack("<i", gph_raw[0])[0] if gph_raw else None
-        return {"speed": speed, "gph": gph}
+        speed_percent = abs(speed) / 10 if speed is not None else None
+        return {"speed": speed, "speed_percent": speed_percent, "gph": gph}
 
     # ---- light schedule (Radion/XR15 etc.) --------------------------------
 

{python_mobius-0.1.1.dist-info → python_mobius-0.1.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-mobius
-Version: 0.1.1
+Version: 0.1.2
 Summary: Reverse-engineered Python client for the Mobius BLE protocol (EcoTech Marine VorTech/Radion, AquaIllumination, Neptune Systems, NYOS)
 Project-URL: Homepage, https://code.r3pek.org/r3pek/python-mobius
 Project-URL: Documentation, https://code.r3pek.org/r3pek/python-mobius/src/branch/main/documentation
@@ -24,6 +24,7 @@ Requires-Python: >=3.9
 Requires-Dist: bleak>=0.21
 Provides-Extra: dev
 Requires-Dist: bleak-retry-connector>=3.0; extra == 'dev'
+Requires-Dist: pytest-asyncio>=0.23; extra == 'dev'
 Requires-Dist: pytest>=7.0; extra == 'dev'
 Provides-Extra: retry
 Requires-Dist: bleak-retry-connector>=3.0; extra == 'retry'

{python_mobius-0.1.1.dist-info → python_mobius-0.1.2.dist-info}/RECORD

@@ -1,14 +1,14 @@
-mobius/__init__.py,sha256=JK6iAhOfAJ1LBf2tKUi3OQY7NMNwOudyyExjYKbehNU,3381
+mobius/__init__.py,sha256=2TkA717bDRzec96psyBxp1NEUi12azr93DxI0iYSfrY,3381
 mobius/cli.py,sha256=uhGZIDrhpTp8LkTGOl4BZVQWgJYHbaIVH2HzX3RFCCs,2642
 mobius/constants.py,sha256=wKrWU-QnqMuWuvczHwt3ha3sPw6D9NNK-kIhxJcTbHA,14702
 mobius/crc.py,sha256=WwIfUXGN_UKdsv-_Q303-L1-L1Rc1fvJnB5176ftzqU,2669
-mobius/device.py,sha256=VeBBT2z0XyqwdnUxw56GrG_ZlVe5Oq05tF-m4z-rMDY,20805
+mobius/device.py,sha256=3kR34RLQYd3lbqG8HeRC5HMvD7k5A6XpBfu_eyASNVw,22779
 mobius/discovery.py,sha256=4MjGwpd0bUK-koiZ-a2Cvq-gCTImAKAQ4lZ-9vOge3Q,2290
 mobius/frame.py,sha256=zG2vh8PceJHD8R43MwZiJnm0Ge1dr8nFcKsJR9_c9LM,6135
 mobius/manufacturer.py,sha256=8-vBzckN8xmsPSMj5EWtOCXenbyQueoY5pnZHePgb_k,1937
 mobius/schedule.py,sha256=kHr684TIKPZ2BplIySoeD-Yn1Xa_2bE6rfGUs-waWMQ,7483
-python_mobius-0.1.1.dist-info/METADATA,sha256=m9dnMFZU2-Qdq_kLSpxPqoYE05Ac3E7NhtSjSmwjXNo,5907
-python_mobius-0.1.1.dist-info/WHEEL,sha256=lCkmxWfQsSc9CfIClYeavTdQeEX2toPqufh9gI35EQA,87
-python_mobius-0.1.1.dist-info/entry_points.txt,sha256=Q2hlcek-bWm70zvd12v32essflPgC2su-dKDHAqbPoE,48
-python_mobius-0.1.1.dist-info/licenses/LICENSE,sha256=7a72Msu2Q-TnoiFxemxEGkwafJGObk1W3rw9hzmyM_Y,17984
-python_mobius-0.1.1.dist-info/RECORD,,
+python_mobius-0.1.2.dist-info/METADATA,sha256=ZfFhhqMEx8S5lyZumY2nMBlmO2bW0HrDtwFi8_gKbbE,5959
+python_mobius-0.1.2.dist-info/WHEEL,sha256=lCkmxWfQsSc9CfIClYeavTdQeEX2toPqufh9gI35EQA,87
+python_mobius-0.1.2.dist-info/entry_points.txt,sha256=Q2hlcek-bWm70zvd12v32essflPgC2su-dKDHAqbPoE,48
+python_mobius-0.1.2.dist-info/licenses/LICENSE,sha256=7a72Msu2Q-TnoiFxemxEGkwafJGObk1W3rw9hzmyM_Y,17984
+python_mobius-0.1.2.dist-info/RECORD,,