Merge pull request #206 from BQSKit/py312-support

Added Python 3.12 Support

Author: edyounis

.github/workflows/tests.yml

@@ -10,7 +10,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        python-version: ['3.8', '3.9', '3.10', '3.11']
+        python-version: ['3.8', '3.9', '3.10', '3.11', '3.12']
         os: [macOS-latest, windows-latest, ubuntu-latest]
 
     runs-on: ${{ matrix.os }}
@@ -19,10 +19,17 @@ jobs:
     - uses: actions/setup-python@v2
       with:
         python-version: ${{ matrix.python-version }}
+
     - name: Upgrade python environment
       run: python -m pip install --upgrade virtualenv setuptools pip
+
     - name: Upgrade test dependencies
-      run: python -m pip install psutil pytest 'hypothesis[zoneinfo]' qiskit
+      run: python -m pip install psutil pytest 'hypothesis[zoneinfo]'
+
+    - name: Upgrade optional test dependencies
+      if: matrix.python-version != '3.12'
+      run: python -m pip install qiskit
+
     - name: Install BQSKit
       env:
         SYSTEM_VERSION_COMPAT: 0

bqskit/compiler/workflow.py

@@ -90,7 +90,7 @@ def name(self) -> str:
     def __str__(self) -> str:
         name_seq = f'Workflow: {self.name}\n\t'
         pass_strs = [
-            f'{i}. {"Workflow: " + p.name if isinstance(p, Workflow) else p}'
+            f'{i}. {'Workflow: ' + p.name if isinstance(p, Workflow) else p}'
             for i, p in enumerate(self._passes)
         ]
         return name_seq + '\n\t'.join(pass_strs)

bqskit/passes/mapping/placement/trivial.py

@@ -20,7 +20,7 @@ async def run(self, circuit: Circuit, data: PassData) -> None:
         model = BasePass.get_model(circuit, data)
         data['placement'] = trivial_placement
 
-        _logger.info(f'Placed qudits on {data["placement"]}')
+        _logger.info(f'Placed qudits on {data['placement']}')
 
         # Raise an error if this is not a valid placement
         sg = model.coupling_graph.get_subgraph(data['placement'])

bqskit/runtime/base.py

@@ -170,7 +170,7 @@ def connect_to_managers(self, ipports: Sequence[tuple[str, int]]) -> None:
                 selectors.EVENT_READ,
                 MessageDirection.BELOW,
             )
-            self.logger.info(f'Registered manager {i} with {num_workers = }.')
+            self.logger.info(f'Registered manager {i} with {num_workers=}.')
             self.total_workers += num_workers
         self.num_idle_workers = self.total_workers
 

pyproject.toml

@@ -9,7 +9,7 @@ requires = [
 legacy_tox_ini = """
 [tox]
 minversion = 3.3.0
-envlist = pre-commit,py38,py39,py310,py311
+envlist = pre-commit,py38,py39,py310,py311,py312
 skip_missing_interpreters = true
 
 [testenv]
@@ -32,7 +32,7 @@ commands =
 skip_install = true
 deps = pre-commit
 commands = pre-commit run --all-files --show-diff-on-failure
-basepython=py311
+basepython=py312
 """
 
 [tool.pytest.ini_options]

setup.py

@@ -45,6 +45,7 @@
         'Programming Language :: Python :: 3.9',
         'Programming Language :: Python :: 3.10',
         'Programming Language :: Python :: 3.11',
+        'Programming Language :: Python :: 3.12',
         'Topic :: Scientific/Engineering',
         'Topic :: Scientific/Engineering :: Mathematics',
         'Topic :: Scientific/Engineering :: Physics',

tests/compiler/compile/test_pam_verify.py

@@ -25,4 +25,4 @@ def test_pam_verify(compiler: Compiler, medium_qasm_file: str) -> None:
     PI = PermutationMatrix.from_qubit_location(out_circuit.num_qudits, pi)
     PF = PermutationMatrix.from_qubit_location(out_circuit.num_qudits, pf)
     exact_error = out_utry.get_distance_from(PF.T @ circuit.get_unitary() @ PI)
-    assert upper_bound_error >= exact_error
+    assert upper_bound_error >= exact_error or abs(upper_bound_error - exact_error) < 5e-7