-
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
Skip transpilation for verbatim circuits #145
Conversation
What do you think about cherry-picking these lines and deleting |
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.
Just a single comment, I'm fine either way.
def _get_gateset(self) -> Optional[set[str]]: | ||
action = self._device.properties.action.get(DeviceActionType.OPENQASM) | ||
if not action: | ||
return None |
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.
Just wonder if we want to keep full control of the gate set (i.e. returning set(BRAKET_TO_QISKIT_NAMES.values())
) instead of letting Qiskit choose the gateset.
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.
to_braket
actually uses _TRANSLATABLE_QISKIT_GATE_NAMES
when gateset=None
, so the gateset is in fact always controlled by the provider.
Summary
Circuits are no longer translated when
verbatim
argument is passed torun
.Details and comments
Also expands the supported gates check to both
BraketLocalBackend
andAWSBraketBackend
.