Skip to content

Update the 'Long-range entanglement with dynamic circuits' tutorial#4987

Open
haimeng-zhang wants to merge 7 commits intoQiskit:mainfrom
haimeng-zhang:update-long-range-entanglement
Open

Update the 'Long-range entanglement with dynamic circuits' tutorial#4987
haimeng-zhang wants to merge 7 commits intoQiskit:mainfrom
haimeng-zhang:update-long-range-entanglement

Conversation

@haimeng-zhang
Copy link
Copy Markdown
Collaborator

Update the 'Long-range entanglement with dynamic circuits' tutorial according to the new template.

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@qiskit-bot
Copy link
Copy Markdown
Contributor

Thanks for contributing to Qiskit documentation!

Before your PR can be merged, it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. Thanks! 🙌

One or more of the following people are relevant to this code:

@henryzou50 henryzou50 self-requested a review April 17, 2026 16:54
Copy link
Copy Markdown
Collaborator

@henryzou50 henryzou50 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice restructure and changes! This looks great and maps cleanly onto the tutorial template. A few suggestions to revise the tutorial itself a bit before merging:

  • Background: the claim about running "multiple LRCX operations in parallel" doesn't match the experiment (which sweeps a single LRCX over distances). Rephrase to reflect the actual comparison, for example:

    In this notebook, we adapt the protocol for IBM Quantum® hardware and benchmark its performance as a function of the control–target separation, comparing it against a unitary SWAP-based baseline.

  • Large-scale Step 1: drop the misleading # Only works for distance larger than 2… comment — distance = 0 is a valid nearest-neighbor CX.
  • Wrap Next steps bullets in <Admonition type="tip" title="Recommendations">…</Admonition> per the template, and use full URLs:
    <Admonition type="tip" title="Recommendations">
    
      - [Benchmark dynamic circuits with cut Bell pairs](https://quantum.cloud.ibm.com/docs/en/tutorials/edc-cut-bell-pair-benchmarking)
      - [Simulation of kicked Ising Hamiltonian with dynamic circuits](https://quantum.cloud.ibm.com/docs/en/tutorials/dc-hex-ising)
    </Admonition>
  • Define MCM on first use, as "mid-circuit measurements (MCMs)".
  • Drop the (i)–(v) sub-step numbering under Steps 1–2. The template's Step 1–4 headings already provide the scaffolding, and right now (i) is orphaned under Step 1 while (ii)–(v) live under Step 2.

@github-project-automation github-project-automation Bot moved this to In Review in Docs Planning Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

3 participants