cgse 0.16.8__tar.gz → 0.16.9__tar.gz

{cgse-0.16.8 → cgse-0.16.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cgse
-Version: 0.16.8
+Version: 0.16.9
 Summary: Generic Common-EGSE: Commanding and monitoring lab equipment
 Author: IvS KU Leuven
 Maintainer-email: Rik Huygen <rik.huygen@kuleuven.be>, Sara Regibo <sara.regibo@kuleuven.be>

{cgse-0.16.8 → cgse-0.16.9}/libs/cgse-common/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "cgse-common"
-version = "0.16.8"
+version = "0.16.9"
 description = "Software framework to support hardware testing"
 authors = [
     {name = "IvS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/libs/cgse-coordinates/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "cgse-coordinates"
-version = "0.16.8"
+version = "0.16.9"
 description = "Reference Frames and Coordinate Transofrmations for CGSE"
 authors = [
     {name = "IvS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/libs/cgse-core/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "cgse-core"
-version = "0.16.8"
+version = "0.16.9"
 description = "Core services for the CGSE framework"
 authors = [
     {name = "IvS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/libs/cgse-core/src/egse/registry/client.py

@@ -329,7 +329,7 @@ class RegistryClient:
             metadata=self._service_info["metadata"],
         )
 
-    def discover_service(self, service_type: str, use_cache: bool = False) -> dict[str, Any] | None:
+    def discover_service(self, service_type: str) -> dict[str, Any] | None:
         """
         Discover a service of the specified type. The service is guaranteed to be healthy at the time of discovery.
 
@@ -341,15 +341,10 @@ class RegistryClient:
 
         Args:
             service_type: Type of service to discover
-            use_cache: Whether to use cached service information
 
         Returns:
             Service information if found, None otherwise
         """
-        # Try to use cache first if enabled
-        if use_cache:
-            self.logger.info("Cache not yet implemented.")
-
         request = {"action": "discover", "service_type": service_type}
 
         response = self._send_request(MessageType.REQUEST_WITH_REPLY, request)
@@ -363,14 +358,13 @@ class RegistryClient:
             self.logger.warning(f"Service discovery failed: {response.get('error')}")
             return None
 
-    def get_service(self, service_id: str | None = None, use_cache: bool = True) -> dict[str, Any] | None:
+    def get_service(self, service_id: str | None = None) -> dict[str, Any] | None:
         """
         Get information about a specific service. When no service_id is given,
         the service_id known to this client will be used.
 
         Args:
             service_id: ID of the service to get [default=None]
-            use_cache: Whether to use cached service information
 
         Returns:
             Service information if found, None otherwise.
@@ -582,10 +576,6 @@ class AsyncRegistryClient:
 
         self._event_handlers = {}
 
-        # Service cache (for discovery)
-        self._service_cache = {}
-        self._service_cache_lock = None
-
         self.req_socket: zmq.asyncio.Socket | None = None
         self.sub_socket: zmq.asyncio.Socket | None = None
         self.hb_socket: zmq.asyncio.Socket | None = None
@@ -634,11 +624,6 @@ class AsyncRegistryClient:
             self.hb_socket.close()
         self.hb_socket = None
 
-    def _get_service_cache_lock(self):
-        if self._service_cache_lock is None:
-            self._service_cache_lock = asyncio.Lock()
-        return self._service_cache_lock
-
     async def _send_request(
         self, msg_type: MessageType, request: dict[str, Any], timeout: float = None
     ) -> dict[str, Any]:
@@ -992,9 +977,6 @@ class AsyncRegistryClient:
 
                         self.logger.debug(f"Received event: {event_type}")
 
-                        # Update service cache based on events
-                        await self._update_cache_from_event(event_type, event)
-
                         # Call registered handlers
                         handlers = self._event_handlers.get(event_type, [])
                         for handler in handlers:
@@ -1022,30 +1004,7 @@ class AsyncRegistryClient:
         self.logger.info("Started event listener task")
         return task
 
-    async def _update_cache_from_event(self, event_type: str, event: dict[str, Any]) -> None:
-        """
-        Update the service cache based on registry events.
-
-        Args:
-            event_type: Type of the event.
-            event: Event data
-        """
-        async with self._get_service_cache_lock():
-            data = event.get("data", {})
-            service_id = data.get("service_id")
-
-            if not service_id:
-                return
-
-            if event_type == "register":
-                service_info = data.get("service_info", {})
-                if service_info:
-                    self._service_cache[service_id] = service_info
-            elif event_type in ("deregister", "expire"):
-                if service_id in self._service_cache:
-                    del self._service_cache[service_id]
-
-    async def discover_service(self, service_type: str, use_cache: bool = True) -> dict[str, Any] | None:
+    async def discover_service(self, service_type: str) -> dict[str, Any] | None:
         """
         Discover a service of the specified type. The service is guaranteed to be healthy at the time of discovery.
 
@@ -1057,27 +1016,11 @@ class AsyncRegistryClient:
 
         Args:
             service_type: Type of service to discover
-            use_cache: Whether to use cached service information
 
         Returns:
             Service information if found, None otherwise
         """
-        # Try to use cache first if enabled
-        if use_cache:
-            async with self._get_service_cache_lock():
-                # Find services of the specified type
-                matching_services = []
-                for service_id, service_info in self._service_cache.items():
-                    if service_info.get("type") == service_type or service_type in service_info.get("tags", []):
-                        matching_services.append(service_info)
-
-                if matching_services:
-                    # Simple load balancing - random selection
-                    import random
-
-                    return random.choice(matching_services)
-
-        # If not found in cache or cache disabled, ask the registry
+
         request = {"action": "discover", "service_type": service_type}
 
         response = await self._send_request(MessageType.REQUEST_WITH_REPLY, request)
@@ -1087,34 +1030,25 @@ class AsyncRegistryClient:
         if response.get("success"):
             service = response.get("service")
 
-            # Update cache
-            if service and "id" in service:
-                async with self._get_service_cache_lock():
-                    self._service_cache[service["id"]] = service
-
             return service
         else:
             self.logger.warning(f"Service discovery failed: {response.get('error')}")
             return None
 
-    async def get_service(self, service_id: str, use_cache: bool = True) -> dict[str, Any] | None:
+    async def get_service(self, service_id: str | None = None) -> dict[str, Any] | None:
         """
-        Get information about a specific service.
+        Get information about a specific service.  When no service_id is given,
+        the service_id known to this client will be used.
 
         Args:
             service_id: ID of the service to get
-            use_cache: Whether to use cached service information
 
         Returns:
             Service information if found, None otherwise
         """
-        # Try to use cache first if enabled
-        if use_cache:
-            async with self._get_service_cache_lock():
-                if service_id in self._service_cache:
-                    return self._service_cache[service_id]
 
-        # If not found in cache or cache disabled, ask the registry
+        service_id = service_id or self._service_id
+
         request = {"action": "get", "service_id": service_id}
 
         response = await self._send_request(MessageType.REQUEST_WITH_REPLY, request)
@@ -1122,11 +1056,6 @@ class AsyncRegistryClient:
         if response.get("success"):
             service = response.get("service")
 
-            # Update cache
-            if service:
-                async with self._get_service_cache_lock():
-                    self._service_cache[service_id] = service
-
             return service
         else:
             self.logger.warning(f"Get service failed: {response.get('error')}")
@@ -1149,12 +1078,6 @@ class AsyncRegistryClient:
         if response.get("success"):
             services = response.get("services", [])
 
-            # Update cache
-            async with self._get_service_cache_lock():
-                for service in services:
-                    if "id" in service:
-                        self._service_cache[service["id"]] = service
-
             return services
         else:
             self.logger.warning(f"List services failed: {response.get('error')}")
@@ -1224,35 +1147,6 @@ class AsyncRegistryClient:
         except Exception as exc:
             self.logger.error(f"Error during cleanup: {exc}")
 
-    @asynccontextmanager
-    async def register_context(self, *args, **kwargs):
-        """
-        Async context manager for service registration.
-
-        Example:
-            async with client.register_context("my-service", "localhost", 8080):
-                # Service is registered
-                await app.start()
-            # Service is automatically deregistered
-        """
-        service_id = await self.register(*args, **kwargs)
-
-        if not service_id:
-            raise RuntimeError("Failed to register service")
-
-        # Start heartbeat and event listener
-        await self.start_heartbeat()
-        await self.start_event_listener()
-
-        try:
-            yield service_id
-        finally:
-            # Clean up
-            await self.stop_event_listener()
-            await self.stop_heartbeat()
-            await self.deregister()
-            # await self.close()  # client shall not be closed by this context manager !!
-
 
 def is_service_registered(service_type: str):
     """Convenience function to check if a service is registered."""

{cgse-0.16.8 → cgse-0.16.9}/libs/cgse-core/src/egse/storage/storage_cs.py

@@ -62,6 +62,8 @@ class StorageControlServer(ControlServer):
 
         multiprocessing.current_process().name = PROCESS_NAME
 
+        self.scheduler: BackgroundScheduler | None = None
+
         self.logger = logger
         self.service_name = PROCESS_NAME
         self.service_type = SERVICE_TYPE
@@ -76,13 +78,6 @@ class StorageControlServer(ControlServer):
 
         self.register_service(service_type=SERVICE_TYPE)
 
-        # NOTE:
-        #     Since the CM CS is started after the SM CS in the normal startup sequence, delay the task to load
-        #     the Setup until after the CM CS has been properly started. We do that now with a delay time of 30s,
-        #     but we might in the future use a function returning a boolean, until...
-        # from egse.confman import is_configuration_manager_active
-        # self.schedule_task(self.device_protocol.controller.load_setup, after=10.0, when=is_configuration_manager_active)
-
     def before_serve(self):
         self.scheduler = BackgroundScheduler(timezone=utc)
         self.scheduler.start()

{cgse-0.16.8 → cgse-0.16.9}/libs/cgse-core/tests/test_registry_service.py

@@ -787,70 +787,3 @@ async def test_client_health_check(client, server):
     """Test health check."""
     health = await client.health_check()
     assert health is True
-
-
-@pytest.mark.asyncio
-async def test_client_caching(client, server):
-    """Test client's service caching."""
-
-    service_id = await client.register("cache-test-service", "localhost", 8080, service_type="cache-test")
-
-    # Look up the service (should populate cache)
-    service = await client.get_service(service_id)
-    assert service is not None
-
-    # Check if it's in the cache
-    async with client._service_cache_lock:
-        assert service_id in client._service_cache
-
-    # Look up by type (should use cache when use_cache=True)
-    with patch.object(client, "_send_request", new_callable=AsyncMock) as mock_send:
-        # First call with use_cache=True should check cache
-        service = await client.discover_service("cache-test", use_cache=True)
-        assert service is not None
-        assert service["id"] == service_id
-
-        # Configure the mock to return a dictionary that matches what _send_request would return
-        mock_send.return_value = {
-            "success": True,
-            "service": {
-                "id": service_id,
-                "name": "cache-test-service",
-                "host": "localhost",
-                "port": 8080,
-                "type": "cache-test",
-            },
-        }
-
-        # Second call with use_cache=False should make a request
-        service = await client.discover_service("cache-test", use_cache=False)
-        assert service is not None
-        assert mock_send.called
-
-
-@pytest.mark.asyncio
-async def test_client_register_context(client, server, req_socket):
-    """Test the register_context context manager."""
-
-    # Use the context manager
-    async with client.register_context(
-        "context-test-service", "localhost", 8080, service_type="context-test"
-    ) as service_id:
-        # Verify the service was registered
-        response = await send_request(
-            MessageType.REQUEST_WITH_REPLY, req_socket, {"action": "get", "service_id": service_id}
-        )
-
-        logger.info(f"----- In test_client_register_context: {response}")
-
-        assert response["success"] is True
-        assert response["service"]["name"] == "context-test-service"
-
-    logger.info("----- In test_client_register_context: after context...")
-
-    # After exiting the context, the service should be deregistered
-    response = await send_request(
-        MessageType.REQUEST_WITH_REPLY, req_socket, {"action": "get", "service_id": service_id}
-    )
-
-    assert response["success"] is False

{cgse-0.16.8 → cgse-0.16.9}/libs/cgse-gui/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "cgse-gui"
-version = "0.16.8"
+version = "0.16.9"
 description = "GUI components for CGSE"
 authors = [
     {name = "Rik Huygen", email = "rik.huygen@kuleuven.be"},

{cgse-0.16.8 → cgse-0.16.9}/projects/generic/cgse-tools/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "cgse-tools"
-version = "0.16.8"
+version = "0.16.9"
 description = "Tools for CGSE"
 authors = [
     {name = "IvS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/projects/generic/keithley-tempcontrol/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "keithley-tempcontrol"
-version = "0.16.8"
+version = "0.16.9"
 description = "Keithley Temperature Control for CGSE"
 authors = [
     {name = "IvS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/projects/generic/lakeshore-tempcontrol/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "lakeshore-tempcontrol"
-version = "0.16.8"
+version = "0.16.9"
 description = "Lakeshore Temperature Control for CGSE"
 authors = [
     {name = "IVS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/projects/generic/symetrie-hexapod/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "symetrie-hexapod"
-version = "0.16.8"
+version = "0.16.9"
 description = "Symetrie Hexapod implementation for CGSE"
 authors = [
     {name = "IvS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/projects/plato/plato-fits/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "plato-fits"
-version = "0.16.8"
+version = "0.16.9"
 description = "FITS Persistence implementation for CGSE"
 authors = [
     {name = "IvS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/projects/plato/plato-hdf5/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "plato-hdf5"
-version = "0.16.8"
+version = "0.16.9"
 description = "HDF5 Persistence sub-class for CGSE"
 authors = [
     {name = "IvS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/projects/plato/plato-spw/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "plato-spw"
-version = "0.16.8"
+version = "0.16.9"
 description = "SpaceWire implementation for CGSE"
 authors = [
     {name = "IvS KU Leuven"}

{cgse-0.16.8 → cgse-0.16.9}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "cgse"
-version = "0.16.8"
+version = "0.16.9"
 description = "Generic Common-EGSE: Commanding and monitoring lab equipment"
 authors = [
     {name = "IvS KU Leuven"}