-
Notifications
You must be signed in to change notification settings - Fork 45
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
Support Parametric Circuits #96
Support Parametric Circuits #96
Conversation
Thank you for this PR @king-p3nguin! Could you please make updates to:
|
Thank you for the review! After fixing the bug, I ran tests locally, and everything appeared to be functioning correctly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the updates! A couple of minor comments
tests/providers/test_adapter.py
Outdated
qiskit_circuit.rz(theta, 0) | ||
braket_circuit = convert_qiskit_to_braket_circuit(qiskit_circuit) | ||
|
||
braket_circuit_ans = Circuit().rz( # pylint: disable=no-member |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we extend the test to include more complex gates like U3 and also mix parametric and non parametric gates?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed that the .u3
method is currently unavailable for QuantumCircuit
. As a solution, I included UGate
in the dictionary. Would this be acceptable?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, thank you!
Co-authored-by: Kshitij Chhabra <kshitijc@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @king-p3nguin! Thank you for this contribution! 🙌
Summary
Details and comments