pyobs-core 2.0.0.dev35__tar.gz → 2.0.0.dev37__tar.gz

{pyobs_core-2.0.0.dev35 → pyobs_core-2.0.0.dev37}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyobs-core
-Version: 2.0.0.dev35
+Version: 2.0.0.dev37
 Summary: robotic telescope software
 Author-email: Tim-Oliver Husser <thusser@uni-goettingen.de>
 License-Expression: MIT

{pyobs_core-2.0.0.dev35 → pyobs_core-2.0.0.dev37}/pyobs/comm/proxy.py

@@ -195,7 +195,13 @@ class Proxy:
         interface: type[Interface],
         timeout: float = 10.0,
     ) -> Any:
-        """Return state immediately if available, otherwise wait for the first update."""
+        """Return state immediately if available, otherwise wait for the first update.
+
+        Returns None on timeout rather than raising -- every caller across the codebase already
+        does `if state is not None else ...` right after calling this, so that's the contract
+        actually being relied on; letting asyncio.wait_for's TimeoutError propagate would just
+        turn "peer hasn't published yet" into an unhandled exception for all of them.
+        """
         if self._state.get(interface) is not None:
             return self._state[interface]
 
@@ -209,6 +215,8 @@ class Proxy:
         await self._comm.subscribe_state(self._client, interface, _notify)
         try:
             await asyncio.wait_for(event.wait(), timeout=timeout)
+        except TimeoutError:
+            pass
         finally:
             await self._comm.unsubscribe_state(self._client, interface, _notify)
 

{pyobs_core-2.0.0.dev35 → pyobs_core-2.0.0.dev37}/pyobs/comm/xmpp/xmppcomm.py

@@ -416,6 +416,17 @@ class XmppComm(Comm):
         if "@" not in client:
             client = f"{client}@{self._domain}/{self._resource}"
 
+        # the client's cache entry is only created once its presence has been processed
+        # (_got_online) -- that can lag slightly behind this module's own startup (e.g. a
+        # background task's very first tick can run before presence has even arrived), so wait
+        # briefly for it instead of failing instantly on a transient "not discovered yet"
+        for _ in range(20):
+            if client in self._interface_cache:
+                break
+            await asyncio.sleep(0.25)
+        else:
+            raise IndexError(f"Client {client} not found.")
+
         # return them from cache
         return await self._interface_cache[client]
 

{pyobs_core-2.0.0.dev35 → pyobs_core-2.0.0.dev37}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pyobs-core"
-version = "2.0.0.dev35"
+version = "2.0.0.dev37"
 description = "robotic telescope software"
 authors = [{ name = "Tim-Oliver Husser", email = "thusser@uni-goettingen.de" }]
 requires-python = ">=3.11"