Merge branch 'dev'

Author: kzscisoft

.pre-commit-config.yaml

@@ -23,7 +23,7 @@ repos:
         args: [--branch, main, --branch, dev]
       - id: check-added-large-files
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.13.3
+    rev: v0.14.3
     hooks:
       - id: ruff
         args: [ --fix, --exit-non-zero-on-fix, "--ignore=C901" ]

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Change Log
 
+## [v2.2.3](https://github.com/simvue-io/client/releases/tag/v2.2.3) - 2025-11-10
+
+- Use `msgpack` for `GridMetrics` in a manner similar to `Metrics`.
+- Fix incorrect setting of global log level.
+- Fix alert duplication in offline mode and other offline fixes.
+
 ## [v2.2.2](https://github.com/simvue-io/client/releases/tag/v2.2.2) - 2025-10-14
 
 - Enforced use of UTC for all datetime recording.

CITATION.cff

@@ -42,6 +42,6 @@ keywords:
   - alerting
   - simulation
 license: Apache-2.0
-commit: 54974cdb71ca4f9ee36541054ef12b853ed99eba
-version: 2.2.2
-date-released: '2025-10-14'
+commit: 4edec13b494f4b6b034d42ad90f33f98a2058b40
+version: 2.2.3
+date-released: '2025-11-10'

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "simvue"
-version = "2.2.2"
+version = "2.2.3"
 description = "Simulation tracking and monitoring"
 authors = [
   {name = "Simvue Development Team", email = "info@simvue.io"}

simvue/api/objects/alert/base.py

@@ -31,6 +31,7 @@ def __init__(self, identifier: str | None = None, **kwargs) -> None:
         """Retrieve an alert from the Simvue server by identifier"""
         self._label = "alert"
         super().__init__(identifier=identifier, **kwargs)
+        self._local_only_args = ["frequency", "pattern", "aggregation"]
 
     def compare(self, other: "AlertBase") -> bool:
         """Compare this alert to another"""

simvue/api/objects/alert/metrics.py

@@ -30,6 +30,12 @@ def __init__(self, identifier: str | None = None, **kwargs) -> None:
         """Connect to a local or remote threshold alert by identifier"""
         self.alert = MetricThresholdAlertDefinition(self)
         super().__init__(identifier, **kwargs)
+        self._local_only_args += [
+            "rule",
+            "window",
+            "metric",
+            "threshold",
+        ]
 
     @classmethod
     def get(
@@ -117,6 +123,13 @@ def __init__(self, identifier: str | None = None, **kwargs) -> None:
         """Connect to a local or remote threshold alert by identifier"""
         self.alert = MetricRangeAlertDefinition(self)
         super().__init__(identifier, **kwargs)
+        self._local_only_args += [
+            "rule",
+            "window",
+            "metric",
+            "range_low",
+            "range_high",
+        ]
 
     def compare(self, other: "MetricsRangeAlert") -> bool:
         """Compare two MetricRangeAlerts"""

simvue/api/objects/artifact/base.py

@@ -54,6 +54,7 @@ def __init__(
         self._label = "artifact"
         self._endpoint = f"{self._label}s"
         super().__init__(identifier=identifier, _read_only=_read_only, **kwargs)
+        self._local_only_args += ["storage", "file_path", "runs"]
 
         # If the artifact is an online instance, need a place to store the response
         # from the initial creation

simvue/api/objects/artifact/file.py

@@ -80,7 +80,9 @@ def new(
         else:
             file_path = pathlib.Path(file_path)
             if snapshot:
-                _user_config = SimvueConfiguration.fetch()
+                _user_config = SimvueConfiguration.fetch(
+                    mode="offline" if offline else "online"
+                )
 
                 _local_staging_dir: pathlib.Path = _user_config.offline.cache.joinpath(
                     "artifacts"

simvue/api/objects/base.py

@@ -33,8 +33,6 @@
 )
 from simvue.api.url import URL
 
-logging.basicConfig(level=logging.INFO)
-
 try:
     from typing import Self
 except ImportError:
@@ -199,6 +197,12 @@ def __init__(
         self._read_only: bool = _read_only
         self._is_set: bool = False
         self._endpoint: str = getattr(self, "_endpoint", f"{self._label}s")
+
+        # For simvue object initialisation, unlike the server there is no nested
+        # arguments, however this means that there are extra keys during post which
+        # need removing, this attribute handles that and should be set in subclasses.
+        self._local_only_args: list[str] = []
+
         self._identifier: str | None = (
             identifier if identifier is not None else f"offline_{uuid.uuid1()}"
         )
@@ -635,6 +639,10 @@ def _post_single(
         if not is_json:
             kwargs = msgpack.packb(data or kwargs, use_bin_type=True)
 
+        # Remove any extra keys
+        for key in self._local_only_args:
+            _ = (data or kwargs).pop(key, None)
+
         _response = sv_post(
             url=f"{self._base_url}",
             headers=self._headers | {"Content-Type": "application/msgpack"},
@@ -672,6 +680,11 @@ def _post_single(
     def _put(self, **kwargs) -> dict[str, typing.Any]:
         if not self.url:
             raise RuntimeError(f"Identifier for instance of {self._label} Unknown")
+
+        # Remove any extra keys
+        for key in self._local_only_args:
+            _ = kwargs.pop(key, None)
+
         _response = sv_put(
             url=f"{self.url}", headers=self._headers, data=kwargs, is_json=True
         )

simvue/api/objects/grids.py

@@ -8,6 +8,7 @@
 """
 
 import http
+import msgpack
 import numpy
 import typing
 
@@ -373,8 +374,9 @@ def _log_values(self, metrics: list[GridMetricSet]) -> None:
 
         _response = sv_post(
             url=f"{self._user_config.server.url}/{self.run_grids_endpoint(self._run_id)}",
-            headers=self._headers,
-            data=metrics,
+            headers=self._headers | {"Content-Type": "application/msgpack"},
+            data=msgpack.packb(metrics, use_bin_type=True),
+            is_json=False,
             params={},
         )