Skip to content

Qiskit examples should not use QR#4780

Open
beckykd wants to merge 77 commits intomainfrom
bd-update-examples
Open

Qiskit examples should not use QR#4780
beckykd wants to merge 77 commits intomainfrom
bd-update-examples

Conversation

@beckykd
Copy link
Collaborator

@beckykd beckykd commented Mar 9, 2026

Part of #4716

Preview changes starting here: https://qiskit.github.io/documentation/pr-4780/docs/guides/install-qiskit

Note that only changed files are built. You will get 404 errors for other files.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@beckykd beckykd marked this pull request as ready for review March 16, 2026 20:00
@beckykd beckykd requested a review from a team March 16, 2026 20:00
@qiskit-bot
Copy link
Contributor

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

Copy link
Collaborator

@jyu00 jyu00 left a comment

Choose a reason for hiding this comment

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

Two additional comments:

  1. This PR replaces a bunch of image files with embedded images with no alt text; not sure if it's intentional.

  2. The diff is too big to leave comments, but the represent-quantum-computers page has

    The default single-qubit gates on ibm_sherbrooke are

    This should be updated to say FakeSherbrooke

"text/plain": [
"<Image src=\"/docs/images/guides/common-parameters/extracted-outputs/dbc652e8-53a4-47a9-a66e-d9c1e5ef07c9-0.svg\" alt=\"Output of the previous code cell\" />"
"<Figure size 1457.85x200.667 with 1 Axes>"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure if it's because I'm in Preview, but I can't see any of the figures on this page.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not sure if it's because I'm in Preview, but I can't see any of the figures on this page.

They show up for me. Maybe try again?

"id": "389bd235-c6c7-4dc0-87c5-272b85cfaae0",
"metadata": {},
"source": [
"#### How the Estimator calculates error\n",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Depending on whether stds is part of the interface, this section is mostly or entirely Qiskit Runtime specific.

"name": "stdout",
"output_type": "stream",
"text": [
"The metadata of the PrimitiveResult is:\n",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Everything below is QRT specific.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Everything below is QRT specific.

Even if we change it to use Statevector? I did delete the last statement about Sampler.

Co-authored-by: Jessie Yu <jessieyu@us.ibm.com>
Co-authored-by: Jessie Yu <jessieyu@us.ibm.com>
abbycross
abbycross previously approved these changes Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants