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

Update circuit helper methods to handle list of size one #73

Closed
wants to merge 2 commits into from

Conversation

jplewa
Copy link

@jplewa jplewa commented Dec 17, 2021

Summary

Updates the qiskit_to_ionq and qiskit_circ_to_ionq_circ helper methods to support a single element, or a list of size one, when being invoked.

Details and comments

This is in accordance with the changes introduced in #71. Support for multi-circuit experiments should probably be added to these methods when #70 is addressed.

@jplewa
Copy link
Author

jplewa commented Dec 17, 2021

Upon further consideration, I'm not sure if these changes are necessary. The issues that I described in #70 and microsoft/azure-quantum-python#200 do play out in the form of qiskit_circ_to_ionq_circ errors, but I think it's just because IonQBackend in azure-quantum doesn't handle circuit lists properly. The proper fix would be to update IonQBackend instead. These helper methods are probably fine the way they are now, unless anyone finds the changes useful in any way ;)

@jplewa jplewa closed this Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant