-
Notifications
You must be signed in to change notification settings - Fork 86
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
feature: add Workspace param to AzureQuantumProvider and AzureQuantumService + Q# upgrade #539
feature: add Workspace param to AzureQuantumProvider and AzureQuantumService + Q# upgrade #539
Conversation
@@ -182,6 +185,10 @@ | |||
input_data_format = kwargs.pop("input_data_format", "qir.v1") | |||
output_data_format = kwargs.pop("output_data_format", self._qir_output_data_format()) | |||
content_type = kwargs.pop("content_type", "qir.v1") | |||
# setting UserAgent header to indicate Q# submission | |||
# TODO: this is a temporary solution. We should be setting the User-Agent header |
Check notice
Code scanning / devskim
A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note
|
||
# Also add all possible shots options into input_params to make sure | ||
# that all backends covered. | ||
# TODO: Double check all backends for shots options in order to remove this extra check. |
Check notice
Code scanning / devskim
A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note
…Service + Q# lang
…object (#542) * Introduce the "shots" parameter to 'sumbit' function of the 'Target' object * Remove num_shots from kwargs. * Add warning on passing shots for estimator * Add 'shots' option for qiskit backends. * Refactor target shots option * The num_shots option for a target class now is checked by a helper function. * New warning is printed on passing shots and num_shots together. * Allow provider-specific shots options. * Fix pascal target * add tests. * Add back all possible shots params --------- Co-authored-by: ArturKamalov <v-akamalov@microsoft.com> Co-authored-by: kikomiss <144282031+kikomiss@users.noreply.github.com>
* Remove aio modules. --------- Co-authored-by: ArturKamalov <v-akamalov@microsoft.com> Co-authored-by: kikomiss <144282031+kikomiss@users.noreply.github.com> Co-authored-by: Kirill Komissarov <kikomiss@microsoft.com>
* Add warning when not using shots parameter. * chore: changed warning verbiage + added missing test recordings --------- Co-authored-by: ArturKamalov <v-akamalov@microsoft.com> Co-authored-by: Kirill Komissarov <kikomiss@microsoft.com>
620120f
to
1bb1d96
Compare
Changes:
AzureQuantumProvider
andAzureQuantumService
withresouce_id
andlocation
parameters.microsoft.quantum-results.v1
job output formatshots
parameter to Qiskit'sbackend.run()
, Passthrough'starget.submit()
and toestimate_cost()
to unify user-experience across backendsazure.quantum.aio
module