A couple of bug fixes including threading issues

Author: kzscisoft

pyproject.toml

@@ -88,12 +88,15 @@ dev = [
 ]
 
 [tool.ruff]
-extend-exclude = ["tests", "examples", "notebooks"]
+extend-exclude = ["tests", "examples", "notebooks", "simvue/pynvml.py"]
+preview = true
 
 [tool.ruff.lint]
+ignore = ["COM812", "PLR0904", "PLR6301", "D203", "D212"]
+dummy-variable-rgx = "^_+$"
 extend-select = [
-  "C901",
-  "T201",
+  "C90",
+  "D417",
   "E",
   "F",
   "W",
@@ -104,6 +107,33 @@ extend-select = [
   "ARG",
   "RUF",
   "RET",
+  "S",
+  "BLE",
+  "FBT",
+  "COM",
+  "A",
+  "C4",
+  "DTZ",
+  "ICN",
+  "G",
+  "PIE",
+  "T20",
+  "INP",
+  "PYI",
+  "PT",
+  "Q",
+  "RSE",
+  "SLOT",
+  "TID",
+  "TC",
+  "ARG",
+  "I",
+  "D417",
+  "ERA",
+  "PL",
+  "UP",
+  "FURB",
+  "N",
 ]
 
 [tool.ruff.lint.mccabe]

simvue/api/__init__.py

@@ -1,5 +1,4 @@
-"""
-Simvue API
+"""Simvue API.
 ==========
 
 Module contains methods for interacting with a Simvue server

simvue/api/objects/__init__.py

@@ -1,5 +1,4 @@
-"""
-Simvue API Objects
+"""Simvue API Objects.
 ==================
 
 The following module defines objects which provide exact representations

simvue/api/objects/administrator/tenant.py

@@ -9,7 +9,9 @@
     from typing import Self, override
 except ImportError:
     from typing_extensions import Self, override
+
 import datetime
+import typing
 from collections.abc import Generator
 
 import pydantic
@@ -34,7 +36,7 @@ def __init__(
         server_token: pydantic.SecretStr | None = None,
         **kwargs,
     ) -> None:
-        """Initialise a Tenant
+        """Initialise a Tenant.
 
         If an identifier is provided a connection will be made to the
         object matching the identifier on the target server.
@@ -52,9 +54,13 @@ def __init__(
             token for alternative server, default None
         **kwargs : dict
             any additional arguments to be passed to the object initialiser
+
         """
         super().__init__(
-            identifier, server_url=server_url, server_token=server_token, **kwargs
+            identifier,
+            server_url=server_url,
+            server_token=server_token,
+            **kwargs,
         )
 
     @override
@@ -124,7 +130,7 @@ def get(
         offset: pydantic.NonNegativeInt | None = None,
         server_url: str | None = None,
         server_token: pydantic.SecretStr | None = None,
-        **kwargs,
+        **kwargs: typing.Any,
     ) -> Generator[tuple[str, Self | None]]:
         """Retrieve tenants from the server.
 
@@ -138,6 +144,8 @@ def get(
             alternative server URL, default None
         server_token : str | None, optional
             token for alternative server, default None
+        **kwargs : Any
+            additional arguments for retrieval
 
         Yields
         ------
@@ -147,86 +155,95 @@ def get(
         Returns
         -------
         Generator[tuple[str, Tenant | None]]
+
         """
         # Currently no tenant filters
         _ = kwargs.pop("filters", None)
         return super().get(
-            count=count, offset=offset, server_url=server_url, server_token=server_token
+            count=count,
+            offset=offset,
+            server_url=server_url,
+            server_token=server_token,
         )
 
     @property
     def name(self) -> str:
-        """Retrieve the name of the tenant"""
+        """Retrieve the name of the tenant."""
         return self._get_attribute("name")
 
     @name.setter
     @write_only
     @pydantic.validate_call
     def name(self, name: str) -> None:
-        """Change name of tenant"""
+        """Change name of tenant."""
         self._staging["name"] = name
 
     @property
     @staging_check
     def is_enabled(self) -> bool:
-        """Retrieve if tenant is enabled"""
+        """Retrieve if tenant is enabled."""
         return self._get_attribute("is_enabled")
 
     @is_enabled.setter
     @write_only
     @pydantic.validate_call
     def is_enabled(self, is_enabled: bool) -> None:
-        """Enable/disable tenant"""
+        """Enable/disable tenant."""
         self._staging["is_enabled"] = is_enabled
 
     @property
     @staging_check
     def max_request_rate(self) -> int:
-        """Retrieve the tenant's maximum request rate"""
+        """Retrieve the tenant's maximum request rate."""
         return self._get_attribute("max_request_rate")
 
     @max_request_rate.setter
     @write_only
     @pydantic.validate_call
     def max_request_rate(self, max_request_rate: int) -> None:
-        """Update tenant's maximum request rate"""
+        """Update tenant's maximum request rate."""
         self._staging["max_request_rate"] = max_request_rate
 
     @property
     @staging_check
     def max_runs(self) -> int:
-        """Retrieve the tenant's maximum runs"""
+        """Retrieve the tenant's maximum runs."""
         return self._get_attribute("max_runs")
 
     @max_runs.setter
     @write_only
     @pydantic.validate_call
     def max_runs(self, max_runs: int) -> None:
-        """Update tenant's maximum runs"""
+        """Update tenant's maximum runs."""
         self._staging["max_runs"] = max_runs
 
     @property
     @staging_check
     def max_data_volume(self) -> int:
-        """Retrieve the tenant's maximum data volume"""
+        """Retrieve the tenant's maximum data volume."""
         return self._get_attribute("max_data_volume")
 
     @max_data_volume.setter
     @write_only
     @pydantic.validate_call
     def max_data_volume(self, max_data_volume: int) -> None:
-        """Update tenant's maximum data volume"""
+        """Update tenant's maximum data volume."""
         self._staging["max_data_volume"] = max_data_volume
 
     @property
     def created(self) -> datetime.datetime | None:
-        """Set/retrieve created datetime for the run.
+        """Set/retrieve created datetime in UTC for the run.
 
         Returns
         -------
         datetime.datetime
+
         """
         _created: str | None = self._get_attribute("created")
         return (
-            datetime.datetime.strptime(_created, DATETIME_FORMAT) if _created else None
+            datetime.datetime.strptime(_created, DATETIME_FORMAT).astimezone(
+                datetime.UTC,
+            )
+            if _created
+            else None
         )