Image issues in transpiler stages topic

[beckykd]

URL to the relevant documentation

https://docs.quantum.ibm.com/transpile/transpiler-stages

Select all that apply

• [ ] typo
• [ ] code bug
• [ ] out-of-date content
• [ ] broken link
• [X] other

Describe the fix.

There are a couple of issues with images on this page.

• [ ] The first image is in dark mode.
• [ ] transpiler-11 is in there twice. I'm pretty sure the one after "The transpilation output varies due to the stochastic SWAP mapper. Therefore, the numbers below will likely change each time you run the code." is supposed to be a different image.

[beckykd]

@abbycross or @kaelynj Can one of you help me figure this out?

[abbycross]

I suspect that the duplicate image Becky mentions above (see screenshot) can be removed altogether, unless @kaelynj do you recall if an image is intentional in this spot?

[kaelynj]

I used the same image for these two examples since they prepare the same example state. Though I see the point about having the same image twice. In the first part where it talks about the routing stage it might be better to instead add in a graph image of the hardware qubits that were assigned after the trivial_layout method was selected using plot_circuit_layout() from the qiskit.visualization module.

[abbycross]

I tried it out, using example lines of code from the API, and got this -- @kaelynj does this look like what you'd expect? 

This is the code I added to the cell

new_circ_lv1 = transpile(ghz, backend=backend, optimization_level=1)
plot_circuit_layout(new_circ_lv1, backend)

[kaelynj]

Yep! This shows the point that section is making about the need to select hardware qubits in a smarter way. The text might need to change a bit to reflect the different figure. Do you want/need me to make those changes?

[abbycross]

Oh my gosh I can't believe I got it right 🤣 If you could make those text changes, that would be wonderful. I'll open a PR and add the new image to it.

[beckykd]

Thanks @kaelynj and @abbycross !

[beckykd]

I used the same image for these two examples since they prepare the same example state. Though I see the point about having the same image twice. In the first part where it talks about the routing stage it might be better to instead add in a graph image of the hardware qubits that were assigned after the trivial_layout method was selected using plot_circuit_layout() from the qiskit.visualization module.

If it's not a mistake, it's fine with me. I just wanted to make sure :)