rename debiasing + backend #124
Conversation
| SYMMETRIZATION = {"symmetrization": True} | ||
| NO_SYMMETRIZATION = {"symmetrization": False} | ||
| DEBIASING = {"debias": True} | ||
| NO_DEBIASING = {"debias": False} |
There was a problem hiding this comment.
I don't like that this requires SDK-side updates if we offer new error mitigation settings.
Is it possible to pass in an arbitrary JSON object for error_mitigation ? Can we support that?
There was a problem hiding this comment.
im fine with that but then we need to have a source of clear documentation of error mitigation settings that we support - that could be https://docs.ionq.com/#tag/jobs under error_mitigation. ill add that as well to the readme
There was a problem hiding this comment.
Ping me on the docs change if you don't mind; mostly curious on the process for rolling those out
qiskit_ionq/ionq_client.py
Outdated
| self, | ||
| job_id: str, | ||
| sharpen: Optional[bool] = None, | ||
| extra_request_payload: Optional[dict] = None, |
There was a problem hiding this comment.
ah, sorry... when I saw that this was updating a dict I thought it was part of the jobs payload. extra_query_params does seem more appropriate (sorry to back and forth so much, this field being optional means folks will need to use its name to reference it so I'm a bit fixated on it at the moment).
Summary
Renames symmetrization to debiasing, updates the v0.3 backend naming scheme, and adds a
extra_request_payloadfor specifying post / get parametersDetails and comments