Fixes bad session management logic. Adds deduplication_id default management. Minor fixes to tests, as well as added coolname dependency.

Author: smaugdela

pennylane_scaleway/aer_device.py

@@ -199,7 +199,7 @@ def execute(
     ) -> List:
 
         if not self._session_id:
-            self.start()
+            raise RuntimeError("No active session. Please instanciate the device using a context manager, or call start() first. You can also attach to an existing deduplication_id.")
 
         if isinstance(circuits, QuantumScript):
             circuits = [circuits]

pennylane_scaleway/aqt_device.py

@@ -242,7 +242,7 @@ def execute(
         execution_config: ExecutionConfig | None = None,
     ) -> List:
         if not self._session_id:
-            self.start()
+            raise RuntimeError("No active session. Please instanciate the device using a context manager, or call start() first. You can also attach to an existing deduplication_id.")
 
         if isinstance(circuits, QuantumScript):
             circuits = [circuits]

pennylane_scaleway/scw_device.py

@@ -126,12 +126,14 @@ def start(self) -> str:
         """
         if not self._session_id:
             self._session_id = self._platform.start_session(**self._session_options)
+
         return self._session_id
 
     def stop(self):
         """
         Stops the currently running session on the Scaleway platform. Raises an error if no session is running.
         """
+
         if self._session_id:
             self._platform.stop_session(self._session_id)
             self.tracker.reset()

requirements.txt

@@ -1,2 +1,3 @@
+coolname
 pennylane
 tenacity

setup.py

@@ -32,7 +32,7 @@
 
 setup(
     name="pennylane-scaleway",
-    version="0.2.4",
+    version="0.2.5",
     project_urls={
         "Documentation": "https://www.scaleway.com/en/quantum-as-a-service/",
         "Source": "https://github.com/scaleway/pennylane-scaleway",

tests/Makefile

@@ -7,7 +7,7 @@ install-pennylane-scaleway: install-dependencies
 .PHONY: install-dependencies
 install-dependencies:
 	pip install --upgrade pip
-	pip install -r ../requirements.txt
+	pip install -Ur ../requirements.txt
 
 .PHONY: install-tests-dependencies
 install-tests-dependencies:

tests/requirements.txt

@@ -1,4 +1,3 @@
 pytest
 pytest-progress
-scipy==1.16.2
-scipy-openblas32==0.3.30.0.4
+scipy

tests/test_aer_device.py

@@ -22,9 +22,13 @@
 # Credentials
 SCW_PROJECT_ID = os.environ["SCW_PROJECT_ID"]
 SCW_SECRET_KEY = os.environ["SCW_SECRET_KEY"]
-SCW_BACKEND_NAME = os.getenv("SCW_BACKEND_NAME", "EMU-AER-16C-128M")
 SCW_API_URL = os.getenv("SCW_API_URL")
 
+if SCW_SECRET_KEY in ["fake-token", ""]:
+    SCW_BACKEND_NAME = "EMU-AER-LOCAL"
+else:
+    SCW_BACKEND_NAME = "EMU-AER-16C-128M"
+
 SHOTS = 4096
 EPSILON = 0.2  # High because we test noisy devices too.
 

tests/test_scaleway_devices.py

@@ -171,7 +171,7 @@ def circuit_probs():
             qml.CNOT(wires=[0, 1])
             return qml.probs(wires=[0, 1])
 
-    probs = circuit_probs()
+        probs = circuit_probs()
 
     # Result should be |00> and |11>
     assert isinstance(probs, np.ndarray)
@@ -305,3 +305,4 @@ def objective_fn(x: np.ndarray) -> float:
         assert np.allclose(
             final_prob_1, 1.0, atol=0.2
         ), f"Expected P(|1>) ~ 1.0, got {final_prob_1}"
+