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

[AQT] Vendor Transformer adoption #5128

Closed
MichaelBroughton opened this issue Mar 22, 2022 · 0 comments · Fixed by #5563
Closed

[AQT] Vendor Transformer adoption #5128

MichaelBroughton opened this issue Mar 22, 2022 · 0 comments · Fixed by #5563
Labels
area/transformers kind/task A task that's part of a larger effort

Comments

@MichaelBroughton
Copy link
Collaborator

Summarize the task
With broader changes to devices (#4744) and optimizers/trasnformers (#4722), vendors might need to update device decomposition to use the newest transformer APIs. At first glance it looks like:


cirq-aqt uses cirq native ion devices which have had their decompose_operation method deprecated in favor of cirq.ConvertToIonGates():

def get_aqt_device(num_qubits: int) -> Tuple[cirq.IonDevice, List[cirq.LineQubit]]:

which is defined here:

class ConvertToIonGates:

Should we move this to cirq-aqt and convert to a transformer to try and consolidate some code here. WDYT @tanujkhattar and @ma5x @pschindler @alfrisch


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/transformers kind/task A task that's part of a larger effort
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants