-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Noiseaware CSPLayout pass #5075
Conversation
This comment has been minimized.
This comment has been minimized.
It could be nice to have some performance/benchmarking data, if it's not too hard to get. |
Thanks for your feedback. I took a look into the other layouters and think it is not easy to compare them. Here you find some benchmarking comparison: As a benchmark I took the problem of simulating the time evolution of an Ising Chain. The calculation was run on ibmq_bogota using three qubits. Three different layouting methods were used and compared to the simulator. At first the classical CSPLayout (dev), the CSPLayoutNoise (dev_ns) as implemented here and a more advanced CSPLayoutAdvancedNoise (dev_avns) as described in slack were used. The circuit is mapped to different physical qubits (0, 1, 2 in case 1 and 2, 3, 4 in the other). The plot shows the probability of measuring the state 111, starting with an initial state of 000 after a certain time. Additionally, the average of five "independent" runs have been taken. This shows that the noise aware versions perform better and are comparable. |
…ut_noiseaware Merge upstream/master for updated qiskit-terra version.
After some chat with team and OP, we agree with the following:
|
before this PR, we need to merge #6263. Therefore, |
Just checking, what's the status of this PR? Why isn't it merged? Still waiting for #6263 ? |
In my opinion this merge request can be closed. As a follow-on of this work we (@1ucian0, @georgios-ts and I) investigated the vf2 method for layout allocation. With merge requests #6620, #7213 and #7276 this has now become a part of qiskit. |
Luciano told me about the VF2 Layout work. Nice! Thanks for closing the PR as well. |
Summary
Refactored the class CSPLayout and created a new class derived of CSPLayout which adds noise awareness.
Closes #5052
Details and comments