pyobs-core 2.0.0.dev38__tar.gz → 2.0.0.dev40__tar.gz

{pyobs_core-2.0.0.dev38 → pyobs_core-2.0.0.dev40}/PKG-INFO

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

{pyobs_core-2.0.0.dev38 → pyobs_core-2.0.0.dev40}/pyobs/application.py

@@ -10,7 +10,7 @@ from typing import Any, TypedDict
 
 import yaml
 
-from pyobs.modules import Module
+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
@@ -154,7 +154,9 @@ class Application:
         from pyobs.utils.logging.context import module_name
 
         log = logging.getLogger(__name__)
-        module_name.set(Path(self._config).stem)
+
+        # set (maybe temporary) module name to filename without leading underscores
+        module_name.set(Path(self._config).stem.lstrip("_"))
 
         # load config
         log.info("Loading configuration from %s...", self._config)
@@ -182,6 +184,23 @@ class Application:
         log.info("Creating module from class %s...", klass.__name__)
         self._module = get_object(cfg, Module)
 
+        # a config file whose stem doesn't match its own module's name silently splits
+        # PYOBS_MODULE log tagging in two: logging that falls through both execute()
+        # (module.py) and BackgroundTask (background_task.py) stays on the filename set
+        # above, while logging covered by either of those uses the module's real
+        # Comm-derived name -- refuse to start rather than let that drift run undetected.
+        # MultiModule is exempt: its own .name is always the fixed placeholder "multi"
+        # (module.py), never meant to match a filename -- each child module is already
+        # correctly distinguished via execute()/BackgroundTask.
+        if not isinstance(self._module, MultiModule):
+            config_stem = Path(self._config).stem
+            if self._module.name != config_stem:
+                raise ValueError(
+                    f"Config file stem ({config_stem!r}) does not match module's own name "
+                    f"({self._module.name!r}). Rename the config file or fix the module's "
+                    f"comm configuration so they match."
+                )
+
     def run(self) -> None:
         """Run app."""
 

{pyobs_core-2.0.0.dev38 → pyobs_core-2.0.0.dev40}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pyobs-core"
-version = "2.0.0.dev38"
+version = "2.0.0.dev40"
 description = "robotic telescope software"
 authors = [{ name = "Tim-Oliver Husser", email = "thusser@uni-goettingen.de" }]
 requires-python = ">=3.11"
@@ -69,7 +69,7 @@ dev = [
     "types-pytz>=2025.2.0.20250326,<2026",
     "types-PyYAML>=6.0.12.20250402,<7",
     "types-requests>=2.32.0.20250328,<3",
-    "types-setuptools>=80.3.0.20250505,<83",
+    "types-setuptools>=80.3.0.20250505,<84",
     "types-toml>=0.10.8.20240310,<0.11",
     "types-tornado>=5.1.1,<6",
     "pytest>=8.3.5,<10",