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

basis translation method based on unitary synthesis #4589

Closed
ajavadia opened this issue Jun 18, 2020 · 0 comments · Fixed by #4763
Closed

basis translation method based on unitary synthesis #4589

ajavadia opened this issue Jun 18, 2020 · 0 comments · Fixed by #4763
Assignees
Labels
status: pending PR It has one or more PRs pending to solve this issue type: feature request New feature or request
Milestone

Comments

@ajavadia
Copy link
Member

As a follow-up to #4446, we should have another method for basis translation that relies on OneQubitEulerDecomposer and TwoQubitBasisDecomposer.

There are some advantages to this, the main one being that the decompositions are algorithmic and can cover a wider range of basis. For example if someone comes up with some esoteric two-qubit gate as the native gate, this code will handle it whereas the EquivalenceLibrary is only as good as its populated content.

This has to be done after all 3+ qubit gates are decomposed. Then I think a sequence of 1-qubit and 2-qubit block collection passes followed by invoking those synthesizers will do the job.

It remains to be seen which basis translation method works best and it's something we will need to benchmark. But this empowers us to do so.

@ajavadia ajavadia added the type: feature request New feature or request label Jun 18, 2020
@ajavadia ajavadia added this to the 0.15 milestone Jun 18, 2020
@kdk kdk added the status: pending PR It has one or more PRs pending to solve this issue label Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: pending PR It has one or more PRs pending to solve this issue type: feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants