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

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

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

{pyobs_core-2.0.0.dev37 → pyobs_core-2.0.0.dev39}/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
@@ -182,6 +182,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.dev37 → pyobs_core-2.0.0.dev39}/pyobs/images/image.py

@@ -2,12 +2,14 @@ from __future__ import annotations
 
 import copy
 import io
+import warnings
 from typing import Any, TypeVar, cast
 
 import numpy as np
 import numpy.typing as npt
 from astropy.io import fits
 from astropy.io.fits import ImageHDU, table_to_hdu
+from astropy.io.fits.verify import VerifyWarning
 from astropy.nddata import CCDData, StdDevUncertainty
 from astropy.table import Table
 from numpy.typing import NDArray
@@ -351,8 +353,13 @@ class Image:
             hdu.name = "RAW"
             hdu_list.append(hdu)
 
-        # write it
-        hdu_list.writeto(f, *args, **kwargs)
+        # write it -- a header value long enough to leave no room for its comment within FITS's
+        # 80-char card limit is expected (header entries come from many different modules we
+        # don't control); astropy already truncates the comment to comply and leaves the value
+        # (the actual data) untouched, so the warning has nothing actionable to tell us
+        with warnings.catch_warnings():
+            warnings.simplefilter("ignore", VerifyWarning)
+            hdu_list.writeto(f, *args, **kwargs)
 
     def to_bytes(self) -> bytes:
         """Write to a bytes array and return it."""

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

@@ -1,6 +1,6 @@
 [project]
 name = "pyobs-core"
-version = "2.0.0.dev37"
+version = "2.0.0.dev39"
 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",