pyobs-core 2.0.0.dev76__tar.gz → 2.0.0.dev77__tar.gz

{pyobs_core-2.0.0.dev76 → pyobs_core-2.0.0.dev77}/PKG-INFO

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

{pyobs_core-2.0.0.dev76 → pyobs_core-2.0.0.dev77}/pyobs/application.py

@@ -16,6 +16,7 @@ from pyobs.modules import Module, MultiModule
 from pyobs.object import get_class_from_string, get_object
 from pyobs.utils.config import pre_process_yaml
 from pyobs.utils.logging.context import ModuleNameFilter
+from pyobs.utils.versions import loaded_pyobs_packages
 
 # just init logger with something here, will be overwritten in __init__
 log = logging.getLogger(__name__)
@@ -375,6 +376,18 @@ class Application:
 
             # open module
             log.info("Opening module...")
+
+            # record the loaded pyobs package versions for comm-less consumers (e.g. web-admin
+            # reading the log/journal); both construction paths have finished importing everything
+            # by this point. Guarded like the IERS priming above: metadata scanning can raise on
+            # any installed distribution with broken dist-info, and that's not worth aborting
+            # startup over.
+            try:
+                packages = loaded_pyobs_packages()
+                log.info("Loaded pyobs packages: %s", ", ".join(f"{k}={v}" for k, v in packages.items()))
+            except Exception:
+                log.warning("Could not determine loaded pyobs package versions, continuing anyway.", exc_info=True)
+
             await self._module.startup()
             log.info("Started successfully.")
 

{pyobs_core-2.0.0.dev76 → pyobs_core-2.0.0.dev77}/pyobs/mixins/weatheraware.py

@@ -76,13 +76,14 @@ class WeatherAwareMixin:
             # do we need to park?
             if motion_status not in [MotionStatus.PARKED, MotionStatus.PARKING]:
                 # error state?
-                if is_error_state and not self.__weatheraware_is_error_state:
-                    log.error("Telescope is in error mode, cannot park.")
+                if is_error_state:
+                    # only log once when entering error state, and never try to park
+                    # while it persists -- park() would just raise again every cycle
+                    if not self.__weatheraware_is_error_state:
+                        log.error("Telescope is in error mode, cannot park.")
                 else:
-                    # parking?
-                    if motion_status != MotionStatus.PARKED:
-                        log.warning("Weather is bad, shutting down.")
-                        await msi.park()
+                    log.warning("Weather is bad, shutting down.")
+                    await msi.park()
 
             # save error state
             self.__weatheraware_is_error_state = is_error_state

pyobs_core-2.0.0.dev77/pyobs/utils/versions.py

@@ -0,0 +1,50 @@
+from __future__ import annotations
+
+from collections.abc import Mapping
+from importlib.metadata import PackageNotFoundError, packages_distributions, version
+
+
+def loaded_pyobs_packages(sys_modules: Mapping[str, object] | None = None) -> dict[str, str]:
+    """Return the version of every loaded ``pyobs``-prefixed distribution.
+
+    Builds the top-level import name -> distribution name map from
+    :func:`importlib.metadata.packages_distributions`, keeps only the distributions whose
+    top-level name is present in ``sys_modules`` and whose name starts with ``pyobs``, and reads
+    each version from :func:`importlib.metadata.version`.
+
+    Args:
+        sys_modules: Module-name -> module mapping to inspect. Defaults to ``sys.modules``.
+
+    Returns:
+        Distribution name -> version, sorted by name. A distribution whose version cannot be
+        resolved (``PackageNotFoundError``) is skipped.
+
+    Note:
+        Editable installs (``pip install -e`` / ``uv pip install -e``, i.e. a dev checkout)
+        typically don't populate the file records that ``packages_distributions()`` needs to
+        derive its top-level-name -> distribution mapping, so a package installed that way is
+        silently omitted here. Not fixed since the only consumer (pyobsd/web-admin reading the
+        module's log/journal) only needs this in production, where installs are non-editable.
+    """
+
+    if sys_modules is None:
+        import sys
+
+        sys_modules = sys.modules
+
+    loaded: dict[str, str] = {}
+    for module_name, dists in packages_distributions().items():
+        if module_name not in sys_modules:
+            continue
+        for dist in dists:
+            if not dist.startswith("pyobs"):
+                continue
+            try:
+                loaded[dist] = version(dist)
+            except PackageNotFoundError:
+                continue
+
+    return dict(sorted(loaded.items()))
+
+
+__all__ = ["loaded_pyobs_packages"]

{pyobs_core-2.0.0.dev76 → pyobs_core-2.0.0.dev77}/pyproject.toml

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