Skip to content

Conversation

miamico
Copy link
Collaborator

@miamico miamico commented Jul 3, 2025

This notebook brings substantial updates to the long-range entanglement tutorial to support the novel capabilities. In particular:

  • removed post-processing technique and text
  • removed randomized certification of fidelity and corresponding text
  • limit number of qubits measured to amount currently supported (16)
  • use hellinger fidelity of the bell pair created by the CNOT between the terminal qubits of the chain as measure of performance

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@miamico miamico linked an issue Jul 3, 2025 that may be closed by this pull request
6 tasks
@qiskit-bot
Copy link
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:

Copy link
Collaborator

@abbycross abbycross left a comment

Choose a reason for hiding this comment

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

Thanks!

@miamico
Copy link
Collaborator Author

miamico commented Jul 21, 2025

@abbycross should we merge this one?

@abbycross abbycross added this pull request to the merge queue Jul 21, 2025
Merged via the queue into main with commit 93c48d7 Jul 21, 2025
4 checks passed
@abbycross abbycross deleted the ma/fix-long-range-entanglement branch July 21, 2025 18:34
@jyu00
Copy link
Collaborator

jyu00 commented Jul 21, 2025

I guess I'm a bit too late, but this should also be updated with the same comment I left for #3479 w.r.t early access - i.e. don't use dynamic_circuits=True (because it'll always return an empty list), and explicitly call out that adding IfElseOp & gen3-experimental is only necessary during early access.

@Eric-Arellano
Copy link
Collaborator

I guess I'm a bit too late

It's not yet synced to closed-source, so can be fixed in a follow up :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Update long-range entanglement tutorial with new dynamic circuits
5 participants