Feature/ready for real device (#9)

* added tag to hybrid_job

* modified role

* try setting role with envvar

* try getting defautl role

* added coverage workflow

* moved coveralls integrateion to test workflow

* added token and changed to lcov format for coverage

* upload cov only on python 3.10

* added coverage badge to readme

* updated name of TrackHHL toymodel to reflect number of detectors and particles

* added python version to precommit-autopdate

* updated classiq to latest version

Author: dmark04

.github/workflows/actions.yml

@@ -48,4 +48,11 @@ jobs:
     - name: Install the project dependencies
       run: poetry install
     - name: Run the automated tests (ignore deprecation errors )
-      run: poetry run pytest -m "not requires_auth" -W ignore::DeprecationWarning --cov-report xml --cov=quantum_linear_systems tests/
+      run: poetry run pytest -m "not requires_auth" -W ignore::DeprecationWarning --cov-report=lcov --cov=quantum_linear_systems tests/
+    - name: Upload results to Coveralls (only on Python3.10 job)
+      if: matrix.python-version == '3.10'
+      env:
+        NODE_COVERALLS_DEBUG: 1
+      uses: coverallsapp/github-action@v2
+      with:
+        github-token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/precommit_autoupd.yml

@@ -13,6 +13,8 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-python@v4
+        with:
+          python-version: '3.10'
       - name: Pre-commit autoupdate
         uses: browniebroke/pre-commit-autoupdate-action@main
       - uses: peter-evans/create-pull-request@v5

README.md

@@ -1,3 +1,5 @@
+[![Coverage Status](https://coveralls.io/repos/github/SURFQuantum/qc-quantum-linear-systems/badge.svg?branch=main)](https://coveralls.io/github/SURFQuantum/qc-quantum-linear-systems?branch=main)
+
 # quantum-linear-systems
 Quantum algorithms to solve linear systems of equations.
 

quantum_linear_systems/execution/aws_execution.py

@@ -7,6 +7,7 @@
 from typing import Tuple
 
 import boto3
+from braket.aws import AwsSession
 from braket.devices import Devices
 from braket.jobs import OutputDataConfig
 from braket.jobs.hybrid_job import hybrid_job
@@ -131,14 +132,20 @@ def aws_s3_folder(folder_name: str) -> Tuple[str, str]:
     )
 
     # Define the role ARN for executing the hybrid job (replace with your actual role ARN)
-    role_arn = "arn:aws:iam::123456789012:role/YourBraketHybridJobRole"
+    # role_arn = "arn:aws:iam::815925483357:role/surf-administrator"
+    os.environ[
+        "BRAKET_JOBS_ROLE_ARN"
+    ] = "arn:aws:iam::815925483357:role/src-workspace-AmazonBraketJobsExecutionRole"
+    aws_session = AwsSession()
+    print("Default role", aws_session.get_default_jobs_role())
 
     @hybrid_job(
         device=device_arn,
-        role_arn=role_arn,
+        # role_arn=role_arn,
         output_data_config=output_data_config,
         dependencies="aws_requirements.txt",
         local=args.local,
+        tags=get_tags(),
     )  # choose priority device
     def execute_hybrid_job() -> None:
         # define hybrid job

quantum_linear_systems/implementations/hhl_classiq_implementation.py

@@ -334,7 +334,7 @@ def solve_hhl_classiq(
 
     gen_circ = GeneratedCircuit.parse_raw(circuit_hhl)
     circuit_depth = gen_circ.transpiled_circuit.depth
-    circuit_width = len(gen_circ.analyzer_data.qubits)
+    circuit_width = gen_circ.data.width
     print("depth = ", circuit_depth)
     print("width = ", circuit_width)
 

quantum_linear_systems/toymodels.py

@@ -359,7 +359,10 @@ def __init__(self, num_detectors: int = 3, num_particles: int = 2) -> None:
         matrix_a, vector_b = upscale_pow2(matrix_a, vector_b)
         csol = np.linalg.solve(matrix_a, vector_b)
         super().__init__(
-            name="HEPSimpleHamiltonian", matrix=matrix_a, vector=vector_b, csol=csol
+            name=f"HEPSimpleHamiltonianD{num_detectors}P{num_particles}",
+            matrix=matrix_a,
+            vector=vector_b,
+            csol=csol,
         )