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

Need method for extracting a initial Layout without running PassManager #2405

Closed
nonhermitian opened this issue May 12, 2019 · 7 comments
Closed
Assignees
Labels
type: enhancement It's working, but needs polishing
Milestone

Comments

@nonhermitian
Copy link
Contributor

What is the expected behavior?

I would like to see what the Layout from virtual to physical qubits is without having to construct and run a PassManager. i.e. I want to be able to easily plot something like this:

tokyo_q7_dense

@nonhermitian nonhermitian added the type: enhancement It's working, but needs polishing label May 12, 2019
@nonhermitian nonhermitian added this to the 0.9 milestone May 12, 2019
@ajavadia
Copy link
Member

Even with a PassManager this is a bit unreliable right now, need to resolve #2066 and #1831 first and make all mappers consistent in how they use Layout.

During the transpilation process a few things can happen that can make the above visualization hard to achieve:

  • ancilla allocation: extra qubits may be used as scratch space for swaps
  • final layout vs. initial layout: the final layout of qubits may be permuted (in fact it usually is)

@nonhermitian
Copy link
Contributor Author

Yes, I should have been a bit more specific in my title. I would like to extract the initial layout that is chosen without actually doing a full transpile. i.e. I want to see the difference in qubit selection between DenseLayout and NoiseAdapativeLayout without transpiling both.

@nonhermitian nonhermitian changed the title Need method for extracting a Layout without running PassManager Need method for extracting a initial Layout without running PassManager May 15, 2019
@1ucian0 1ucian0 self-assigned this May 30, 2019
@1ucian0
Copy link
Member

1ucian0 commented Jul 19, 2019

After #2736, the layout is saved as circuit.layout after transpilation. Is this solving this issue?

@1ucian0 1ucian0 added the status: needs information Further information is requested label Jul 19, 2019
@nonhermitian
Copy link
Contributor Author

yep

@1ucian0
Copy link
Member

1ucian0 commented Aug 6, 2019

Reopening, because #2736 was kinda reverted by #2928

@1ucian0 1ucian0 reopened this Aug 6, 2019
@1ucian0 1ucian0 removed the status: needs information Further information is requested label Aug 6, 2019
@1ucian0
Copy link
Member

1ucian0 commented Aug 9, 2019

Is this a 0.9 issue @ajavadia ?

@nonhermitian
Copy link
Contributor Author

This can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement It's working, but needs polishing
Projects
None yet
Development

No branches or pull requests

3 participants