Update AI transpiler introduction tutorial to new template#4968
Update AI transpiler introduction tutorial to new template#4968henryzou50 wants to merge 18 commits into
Conversation
Revised ai-transpiler-introduction.ipynb following the Tutorial_Template structure. - Compare default (SABRE) vs AI transpiler using random circuits with only 2-qubit gates across small-scale (6-25 qubits) and large-scale (26-50 qubits) examples - Use mirror circuits to evaluate transpilation fidelity on both Aer simulator (with depolarizing noise model) and real hardware - Add summary tables with mean/stdev and percentage improvement metrics - Add percentage improvement plots comparing AI vs default transpiler - Include commentary on depth vs gate count trade-offs between the two strategies
|
One or more of the following people are relevant to this code:
|
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
nathanearnestnoble
left a comment
There was a problem hiding this comment.
The large scale section states "step 1-4 compress into single code block" but is in multiple blocks. Would suggest removing "compress into single code block"
Thanks @nathanearnestnoble , I just pushed a change for that and added the reasoning for why we don't compress the large scale example into a single code block for this tutorial. |
kaelynj
left a comment
There was a problem hiding this comment.
Looking pretty good so far! Left some comments to address
Also a side note: @henryzou50 you should have write access to the repo now since you're a regular contributor so you shouldn't need to open PRs through a fork.
Co-authored-by: Kaelyn Ferris <43348706+kaelynj@users.noreply.github.com>
Co-authored-by: Kaelyn Ferris <43348706+kaelynj@users.noreply.github.com>
Co-authored-by: Kaelyn Ferris <43348706+kaelynj@users.noreply.github.com>
Co-authored-by: Kaelyn Ferris <43348706+kaelynj@users.noreply.github.com>
Co-authored-by: Kaelyn Ferris <43348706+kaelynj@users.noreply.github.com>
Co-authored-by: Kaelyn Ferris <43348706+kaelynj@users.noreply.github.com>
Co-authored-by: Kaelyn Ferris <43348706+kaelynj@users.noreply.github.com>
Co-authored-by: Kaelyn Ferris <43348706+kaelynj@users.noreply.github.com>
From Kaelynj's review: - Replace the pandas DataFrame summary with a plain printed table, since the DataFrame did not render correctly on the docs web page - Remove the per-circuit print output from the transpilation loops; only the mean/std summary table is needed - Plot 1 - P(|0...0>) instead of the raw survival probability so the mirror-circuit fidelity results are legible (the raw bars were flat near zero on hardware) - Restore the AI-synthesis coupling-map limitation note in the analysis - Drop "service" from the H1 heading so it matches the frontmatter title - Apply wording suggestions in the fidelity analysis Additional changes: - Refactor summary/plot helpers to work off the list of result dicts and remove the pandas dependency entirely - Reconcile commentary with the re-run results: circuit depth (4, not 5), depth/gate-count percentages, the 10-qubit fidelity discussion, and the large-scale noise-floor wording - Fix the stale "reduced coupling map" description in Step 2 to reflect the actual transpile-to-full-backend + remap_to_contiguous approach - Convert absolute quantum.cloud.ibm.com doc links to relative paths for consistency with other recently updated tutorials - Fix the broken generate_preset_pass_manager reference link and typo in the Background section
|
Thanks so much for the review, @kaelynj I've applied your suggestions:
While I was in there, I also made a few related changes:
Let me know if you'd like any further changes! |
Summary
Revised
ai-transpiler-introduction.ipynbto follow the Tutorial_Template structure, simplifying the tutorial to focus on comparing the default transpiler (SABRE) vs the AI transpiler using random circuits.Key changes from the old notebook:
Tutorial structure: