-
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
Move gateset construction to adapter #146
Conversation
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.
LGTM
gateset: Optional[Iterable[str]] = None, | ||
basis_gates: Optional[ | ||
Iterable[str] | ||
] = None, # TODO: Use backend instead of basis_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.
Note: I like passing list[str] that is simpler than a Backend. One could tweak their circuits for a certain gateset that they'd like to try.
At some point, I think we should expose to_braket and to_qiskit at higher level.
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.
That makes sense; this TODO was motivated by looking at all the work put into constructing a Target
only to never use it.
Summary
Put gateset construction in a more logical place and made constants and methods private as needed.
Details and comments
Also skip gateset construction when running circuits verbatim.