Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open v0.3 API + debiasing #75

Merged
merged 34 commits into from
Jan 2, 2024
Merged
Show file tree
Hide file tree
Changes from 15 commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
fb07c8a
create job with debias
splch Jul 24, 2023
0d01740
get sharpened results
splch Jul 24, 2023
ac9d407
add error_mitigation and sharpen to the end
splch Jul 24, 2023
d54f871
use userwarning
splch Jul 24, 2023
543c885
fix undefined params and sharpen
splch Jul 24, 2023
2386290
reorder args
splch Jul 24, 2023
bdfa904
final arg reorder
splch Jul 24, 2023
2fc6d9b
Merge branch 'master' into v0.3-and-symmetrization
timmysilv Jul 24, 2023
f5503b2
remove unnecessary kwargs pop
splch Jul 24, 2023
a0a77a7
use new fields for tests
splch Aug 3, 2023
e8ad837
add to contributors
splch Aug 7, 2023
1efeb0d
format code + add params
splch Oct 19, 2023
8daacfe
Merge branch 'master' into v0.3-and-symmetrization
splch Oct 19, 2023
ef57f5f
Merge branch 'PennyLaneAI:master' into v0.3-and-symmetrization
splch Oct 30, 2023
be25d2e
Merge branch 'master' into v0.3-and-symmetrization
splch Nov 4, 2023
d4b5d7c
Merge branch 'master' into v0.3-and-symmetrization
lillian542 Dec 15, 2023
075f551
Merge branch 'master' into v0.3-and-symmetrization
lillian542 Dec 18, 2023
10116db
Update CHANGELOG.md
lillian542 Dec 18, 2023
f28f2a9
increase test coverage
lillian542 Dec 18, 2023
ba6dd7e
update changelog
lillian542 Dec 18, 2023
01f6042
add error_mitigation testing
lillian542 Dec 18, 2023
c9750f1
expand on docstrings
lillian542 Dec 18, 2023
6c288f7
black formatting
lillian542 Dec 18, 2023
b3c6f61
update overview doc to refer to docstrings for details
lillian542 Dec 18, 2023
100d6bf
more black formatting
lillian542 Dec 18, 2023
c2a9793
update hyperlink
lillian542 Dec 18, 2023
0bfef67
troubleshoot docstring formatting problem
lillian542 Dec 18, 2023
bed8239
troubleshoot docstring formatting problem 2
lillian542 Dec 18, 2023
27c332f
update changelog and docstrings
lillian542 Dec 18, 2023
b37d18e
black formatting for tests
lillian542 Dec 18, 2023
33ab2da
correct wording
lillian542 Dec 18, 2023
a4712a6
default backend is harmony
timmysilv Jan 2, 2024
7bd4a44
try to please codecov
timmysilv Jan 2, 2024
36b8c96
revert and simplify default target
timmysilv Jan 2, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,11 @@

### Improvements 🛠

* adds a new `error_mitigation` parameter on IonQ job submission so users can
configure strategies
* adds a new `aggregation` parameter on IonQ results to allow getting results
aggregated under the two different methods described in
<https://arxiv.org/pdf/2301.07233.pdf>
lillian542 marked this conversation as resolved.
Show resolved Hide resolved
* Use new `backend` field to specify `qpu`.
[(#81)](https://github.com/PennyLaneAI/PennyLane-IonQ/pull/81)

Expand Down
21 changes: 13 additions & 8 deletions pennylane_ionq/api_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -112,12 +112,16 @@
api_key (str): IonQ cloud platform API key
"""

USER_AGENT = "pennylane-ionq-api-client/0.1"
HOSTNAME = "api.ionq.co/v0.1"
USER_AGENT = "pennylane-ionq-api-client/0.3"
HOSTNAME = "api.ionq.co/v0.3"
BASE_URL = "https://{}".format(HOSTNAME)

def __init__(self, **kwargs):
self.AUTHENTICATION_TOKEN = os.getenv("IONQ_API_KEY") or kwargs.get("api_key", None)
self.AUTHENTICATION_TOKEN = (
kwargs.get("api_key", None)
or os.getenv("PENNYLANE_IONQ_API_KEY")
or os.getenv("IONQ_API_KEY")
)
timmysilv marked this conversation as resolved.
Show resolved Hide resolved
self.DEBUG = False

if "IONQ_DEBUG" in os.environ:
Expand Down Expand Up @@ -196,17 +200,18 @@

return response

def get(self, path):
def get(self, path, params=None):
"""
Sends a GET request to the provided path. Returns a response object.

Args:
path (str): path to send the GET request to
params (dict): parameters to include in the request

Returns:
requests.Response: A response object, or None if no response could be fetched
"""
return self.request(requests.get, url=self.join_path(path))
return self.request(requests.get, url=self.join_path(path), params=params)

def post(self, path, payload):
"""
Expand Down Expand Up @@ -249,7 +254,7 @@
"""
return join_path(self.resource.PATH, path)

def get(self, resource_id=None):
def get(self, resource_id=None, params=None):
"""
Attempts to retrieve a particular record by sending a GET
request to the appropriate endpoint. If successful, the resource
Expand All @@ -262,9 +267,9 @@
raise MethodNotSupportedException("GET method on this resource is not supported")

if resource_id is not None:
response = self.client.get(self.join_path(str(resource_id)))
response = self.client.get(self.join_path(str(resource_id)), params=params)
else:
response = self.client.get(self.resource.PATH)
response = self.client.get(self.resource.PATH, params=params)

Check warning on line 272 in pennylane_ionq/api_client.py

View check run for this annotation

Codecov / codecov/patch

pennylane_ionq/api_client.py#L272

Added line #L272 was not covered by tests
lillian542 marked this conversation as resolved.
Show resolved Hide resolved
self.handle_response(response)

def create(self, **params):
Expand Down Expand Up @@ -337,7 +342,7 @@
def refresh_data(self, data):
"""
Refreshes the instance's attributes with the provided data and
converts it to the correct type.

Check notice on line 345 in pennylane_ionq/api_client.py

View check run for this annotation

codefactor.io / CodeFactor

pennylane_ionq/api_client.py#L345

Raising too general exception: Exception (broad-exception-raised)

Args:
data (dict): A dictionary containing keys and values of data to be stored on the object.
Expand Down
37 changes: 32 additions & 5 deletions pennylane_ionq/device.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"""
This module contains the device class for constructing IonQ devices for PennyLane.
"""
import os, warnings

Check notice on line 17 in pennylane_ionq/device.py

View check run for this annotation

codefactor.io / CodeFactor

pennylane_ionq/device.py#L17

Unused import os (unused-import)
from time import sleep

import numpy as np
Expand Down Expand Up @@ -91,14 +91,26 @@
# and therefore does not support the Hermitian observable.
observables = {"PauliX", "PauliY", "PauliZ", "Hadamard", "Identity"}

def __init__(self, wires, *, target="simulator", gateset="qis", shots=1024, api_key=None):
def __init__(
self,
wires,
*,
target="simulator",
gateset="qis",
shots=1024,
api_key=None,
error_mitigation=None,
sharpen=None,
lillian542 marked this conversation as resolved.
Show resolved Hide resolved
):
if shots is None:
raise ValueError("The ionq device does not support analytic expectation values.")

super().__init__(wires=wires, shots=shots)
self.target = target
self.api_key = api_key
self.gateset = gateset
self.error_mitigation = error_mitigation
self.sharpen = sharpen
self._operation_map = _GATESET_OPS[gateset]
self.reset()

Expand All @@ -107,16 +119,25 @@
self._prob_array = None
self.histogram = None
self.circuit = {
"format": "ionq.circuit.v0",
"qubits": self.num_wires,
"circuit": [],
"gateset": self.gateset,
}
self.job = {
"lang": "json",
"body": self.circuit,
"input": self.circuit,
"target": self.target,
"shots": self.shots,
}
if self.error_mitigation is not None:
self.job["error_mitigation"] = self.error_mitigation

Check warning on line 133 in pennylane_ionq/device.py

View check run for this annotation

Codecov / codecov/patch

pennylane_ionq/device.py#L133

Added line #L133 was not covered by tests
lillian542 marked this conversation as resolved.
Show resolved Hide resolved
if self.job["target"] == "qpu":
self.job["target"] = "qpu.harmony"
warnings.warn(
"The ionq_qpu backend is deprecated. Defaulting to ionq_qpu.harmony.",
UserWarning,
stacklevel=2,
)

@property
def operations(self):
Expand Down Expand Up @@ -190,7 +211,9 @@
if job.is_failed:
raise JobExecutionError("Job failed")

job.manager.get(job.id.value)
params = {} if self.sharpen is None else {"sharpen": self.sharpen}

job.manager.get(resource_id=job.id.value, params=params)

# The returned job histogram is of the form
# dict[str, float], and maps the computational basis
Expand Down Expand Up @@ -287,9 +310,11 @@
wires,
*,
target="qpu",
backend=None,
gateset="qis",
shots=1024,
backend=None,
error_mitigation=None,
sharpen=None,
api_key=None,
):
self.backend = backend
Expand All @@ -301,6 +326,8 @@
gateset=gateset,
shots=shots,
api_key=api_key,
error_mitigation=error_mitigation,
sharpen=sharpen,
)

def generate_samples(self):
Expand Down
8 changes: 6 additions & 2 deletions tests/test_api_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -303,12 +303,16 @@ def mock_raise(exception):
mock_get_response = MockGETResponse(200)

monkeypatch.setattr(
requests, "get", lambda url, timeout, headers: mock_get_response
requests,
"get",
lambda url, params=None, timeout=None, headers=None: mock_get_response,
)
monkeypatch.setattr(
requests,
"post",
lambda url, timeout, headers, data: mock_raise(MockException),
lambda url, data=None, timeout=None, headers=None: mock_raise(
MockException
),
)

client = api_client.APIClient(debug=True, api_key="test")
Expand Down
40 changes: 20 additions & 20 deletions tests/test_device.py
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ def test_shots(self, shots, monkeypatch, mocker, tol):
)
monkeypatch.setattr(Job, "is_complete", True)

def fake_response(self, resource_id=None):
def fake_response(self, resource_id=None, params=None):
"""Return fake response data"""
fake_json = {"histogram": {"0": 1}}
setattr(
Expand Down Expand Up @@ -216,13 +216,13 @@ def mock_submit_job(*args):

dev.apply(tape.operations)

assert dev.job["lang"] == "json"
assert dev.job["body"]["gateset"] == "qis"
assert dev.job["input"]["format"] == "ionq.circuit.v0"
assert dev.job["input"]["gateset"] == "qis"
assert dev.job["target"] == "foo"
assert dev.job["body"]["qubits"] == 1
assert dev.job["input"]["qubits"] == 1

assert len(dev.job["body"]["circuit"]) == 1
assert dev.job["body"]["circuit"][0] == {"gate": "x", "target": 0}
assert len(dev.job["input"]["circuit"]) == 1
assert dev.job["input"]["circuit"][0] == {"gate": "x", "target": 0}

def test_parameterized_op(self, mocker):
"""Tests job attribute several parameterized operations."""
Expand All @@ -239,17 +239,17 @@ def mock_submit_job(*args):

dev.apply(tape.operations)

assert dev.job["lang"] == "json"
assert dev.job["body"]["gateset"] == "qis"
assert dev.job["body"]["qubits"] == 1
assert dev.job["input"]["format"] == "ionq.circuit.v0"
assert dev.job["input"]["gateset"] == "qis"
assert dev.job["input"]["qubits"] == 1

assert len(dev.job["body"]["circuit"]) == 2
assert dev.job["body"]["circuit"][0] == {
assert len(dev.job["input"]["circuit"]) == 2
assert dev.job["input"]["circuit"][0] == {
"gate": "rx",
"target": 0,
"rotation": 1.2345,
}
assert dev.job["body"]["circuit"][1] == {
assert dev.job["input"]["circuit"][1] == {
"gate": "ry",
"target": 0,
"rotation": 2.3456,
Expand All @@ -269,24 +269,24 @@ def mock_submit_job(*args):
GPI2(0.2, wires=[1])
MS(0.2, 0.3, wires=[1, 2])

assert dev.job["lang"] == "json"
assert dev.job["body"]["gateset"] == "native"
assert dev.job["body"]["qubits"] == 3

dev.apply(tape.operations)

assert len(dev.job["body"]["circuit"]) == 3
assert dev.job["body"]["circuit"][0] == {
assert dev.job["input"]["format"] == "ionq.circuit.v0"
assert dev.job["input"]["gateset"] == "native"
assert dev.job["input"]["qubits"] == 3

assert len(dev.job["input"]["circuit"]) == 3
assert dev.job["input"]["circuit"][0] == {
"gate": "gpi",
"target": 0,
"phase": 0.1,
}
assert dev.job["body"]["circuit"][1] == {
assert dev.job["input"]["circuit"][1] == {
"gate": "gpi2",
"target": 1,
"phase": 0.2,
}
assert dev.job["body"]["circuit"][2] == {
assert dev.job["input"]["circuit"][2] == {
"gate": "ms",
"targets": [1, 2],
"phases": [0.2, 0.3],
Expand Down
Loading