👷 Fix minimums nox session (#468)

## Description

This PR fixes the `minimums` nox session by setting a lower bound for
Qiskit. The sessions is currently failing (including in the CI on
`main`) because MQT Predictor is not yet compatible with the latest
Qiskit version.

As a lock-file update was necessary, this PR also makes MQT Predictor
compatible with the latest Qiskit version.

## Checklist:

- [x] The pull request only contains commits that are focused and
relevant to this change.
- [x] ~I have added appropriate tests that cover the new/changed
functionality.~
- [x] ~I have updated the documentation to reflect these changes.~
- [x] ~I have added entries to the changelog for any noteworthy
additions, changes, fixes, or removals.~
- [x] ~I have added migration instructions to the upgrade guide (if
needed).~
- [x] The changes follow the project's style guidelines and introduce no
new warnings.
- [x] The changes are fully tested and pass the CI checks.
- [x] I have reviewed my own code changes.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: burgholzer <burgholzer@me.com>

Author: 3 people

.gitignore

@@ -51,3 +51,5 @@ venv.bak/
 
 .ruff_cache/
 .mypy_cache/
+
+src/**/_version.py

noxfile.py

@@ -11,14 +11,16 @@
 from __future__ import annotations
 
 import argparse
+import contextlib
 import os
 import shutil
+import tempfile
 from typing import TYPE_CHECKING
 
 import nox
 
 if TYPE_CHECKING:
-    from collections.abc import Sequence
+    from collections.abc import Generator, Sequence
 
 
 nox.needs_version = ">=2024.3.2"
@@ -35,6 +37,17 @@
     nox.options.error_on_missing_interpreters = True
 
 
+@contextlib.contextmanager
+def preserve_lockfile() -> Generator[None]:
+    """Preserve the lockfile by moving it to a temporary directory."""
+    with tempfile.TemporaryDirectory() as temp_dir_name:
+        shutil.move("uv.lock", f"{temp_dir_name}/uv.lock")
+        try:
+            yield
+        finally:
+            shutil.move(f"{temp_dir_name}/uv.lock", "uv.lock")
+
+
 @nox.session(reuse_venv=True)
 def lint(session: nox.Session) -> None:
     """Run the linter."""
@@ -82,25 +95,15 @@ def tests(session: nox.Session) -> None:
 @nox.session(reuse_venv=True, venv_backend="uv", python=PYTHON_ALL_VERSIONS)
 def minimums(session: nox.Session) -> None:
     """Test the minimum versions of dependencies."""
-    _run_tests(
-        session,
-        install_args=["--resolution=lowest-direct"],
-        pytest_run_args=["-Wdefault"],
-    )
-    env = {"UV_PROJECT_ENVIRONMENT": session.virtualenv.location}
-    session.run("uv", "tree", "--frozen", env=env)
-    session.run("uv", "lock", "--refresh", env=env)
-
-
-@nox.session(reuse_venv=True, venv_backend="uv", python=PYTHON_ALL_VERSIONS)
-def qiskit(session: nox.Session) -> None:
-    """Tests against the latest version of Qiskit."""
-    _run_tests(
-        session,
-        extra_command=["uv", "pip", "install", "qiskit[qasm3-import] @ git+https://github.com/Qiskit/qiskit.git"],
-    )
-    env = {"UV_PROJECT_ENVIRONMENT": session.virtualenv.location}
-    session.run("uv", "pip", "show", "qiskit", env=env)
+    with preserve_lockfile():
+        _run_tests(
+            session,
+            install_args=["--resolution=lowest-direct"],
+            pytest_run_args=["-Wdefault"],
+        )
+        env = {"UV_PROJECT_ENVIRONMENT": session.virtualenv.location}
+        session.run("uv", "tree", "--frozen", env=env)
+        session.run("uv", "lock", "--refresh", env=env)
 
 
 @nox.session(reuse_venv=True)

pyproject.toml

@@ -30,9 +30,9 @@ dynamic = ["version"]
 
 dependencies = [
     "mqt.bench>=2.0.0",
-    "qiskit!=1.3.2", # 1.3.2 causes a Qiskit error when using the CommutativeInverseCancellation pass, see https://github.com/Qiskit/qiskit/issues/13742
+    "qiskit>=1.3.3,<2.2.0",
     "pytket>=1.29.0", # lowest version that supports the used pytket AutoRebase pass instead of auto_rebase
-    "pytket_qiskit>=0.60.0",
+    "pytket_qiskit>=0.61.0",
     "sb3_contrib>=2.0.0",
     "tqdm>=4.66.0",
     "rich>=12.6.0",

tests/compilation/test_reward.py

@@ -16,17 +16,23 @@
 from mqt.bench import BenchmarkLevel, get_benchmark
 from mqt.bench.targets import get_device
 from qiskit import transpile
-
-if TYPE_CHECKING:
-    from qiskit import QuantumCircuit
-    from qiskit.transpiler import Target
-
-
 from qiskit.circuit.library import CXGate, Measure, XGate
 from qiskit.transpiler import InstructionProperties, Target
 
 from mqt.predictor.reward import crit_depth, esp_data_available, estimated_success_probability, expected_fidelity
 
+try:
+    from qiskit.providers.backend import QubitProperties
+
+    QISKIT_PRE_2_0 = False
+except ImportError:
+    QubitProperties = object
+
+    QISKIT_PRE_2_0 = True
+
+if TYPE_CHECKING:
+    from qiskit import QuantumCircuit
+
 
 @pytest.fixture
 def device() -> Target:
@@ -92,10 +98,16 @@ def make_target(
     if no_qubit_props:
         t.qubit_properties = None
     else:
-        t.qubit_properties = [
-            type("QubitProps", (), {"t1": t1, "t2": t2}),
-            type("QubitProps", (), {"t1": t1, "t2": t2}),
-        ]
+        if QISKIT_PRE_2_0:
+            t.qubit_properties = [
+                type("QubitProp", (), {"t1": t1, "t2": t2}),
+                type("QubitProps", (), {"t1": t1, "t2": t2}),
+            ]
+        else:
+            t.qubit_properties = [
+                QubitProperties(t1=t1, t2=t2),
+                QubitProperties(t1=t1, t2=t2),
+            ]
     return t