Add Cirq and Rigetti content from Azure-Quantum to QDK package #2824
Conversation
ScottCarda-MS
changed the title
ScottCarda-MS
requested review from
billti,
idavis,
minestarks and
swernli
as code owners
| "azure-quantum>=3.5.0", | ||
| "qiskit>=1.2.2,<3.0.0", | ||
| "cirq-core>=1.3.0,<=1.4.1", | ||
| "cirq-ionq>=1.3.0,<=1.4.1", |
There was a problem hiding this comment.
Why do we need a separate cirq-ionq package? Is this similar to the qiskit-ionq dependency @idavis removed (https://github.com/microsoft/azure-quantum-python/pull/694/changes#diff-7be6800e2a7d73f55566f4819bbc22349cf365d410a0d50d3efe746b9c3c6e64)? If so, should we do something similar here?
There was a problem hiding this comment.
(Actually, I see this is a package by cirq itself (https://github.com/quantumlib/Cirq/tree/main/cirq-ionq), so possibly unrelated to the pattern that was used by qiskit-ionq)
There was a problem hiding this comment.
These are the requirements pulled from azure quantum: https://github.com/microsoft/azure-quantum-python/blob/main/azure-quantum/requirements-cirq.txt. @idavis suggested that we reference this dependency directly, though, by having the azure extra depend on azure-quantum[cirq], which is an idea I like as it removes the need for an additional "cirq" specific extra.
Adds the cirq extra,
qdk.azure.cirq, andqdk.azure.target.rigetti.