Skip to content

Consolidate two similar pages (get-qpu-information and qpu-information)#4313

Merged
abbycross merged 132 commits intomainfrom
ajc/consolidate-qpu-info
Mar 5, 2026
Merged

Consolidate two similar pages (get-qpu-information and qpu-information)#4313
abbycross merged 132 commits intomainfrom
ajc/consolidate-qpu-info

Conversation

@abbycross
Copy link
Collaborator

@abbycross abbycross commented Nov 25, 2025

CC: @jyu00

Consolidate two files containing similar/overlapping content: Get backend information with Qiskit and QPU information.

@Eric-Arellano @frankharkins @arnaucasau FYI: when it is time to merge, can you please set up a redirect from qpu-information.mdx to get-qpu-information.ipynb?

Closes #4249

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@abbycross abbycross changed the title Ajc/consolidate qpu info Consolidate two similar pages (get-qpu-information and qpu-information) Nov 26, 2025
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.

Sorry if I reviewed too early... didn't realize it's still in draft!

@abbycross abbycross linked an issue Dec 5, 2025 that may be closed by this pull request
"</Tabs>\n",
"\n",
"<span id=\"native-gates\"></span>\n",
"### Table of supported instructions\n",
Copy link
Collaborator

Choose a reason for hiding this comment

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

What you did is fine, but we can also consolidate a bit by putting both delay and measure_* in the Non-unitary instructions row.

@abbycross
Copy link
Collaborator Author

Not quite ready for final approval - need to convert the tables into screen-reader-accessible structures first (but will take advantage of the lovely new accordion component) 🪗 🪗 🪗

@abbycross
Copy link
Collaborator Author

Ready for final review!

jyu00
jyu00 previously approved these changes Mar 5, 2026
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.

The new accordions look great! Thanks Abby!

test.ipynb Outdated
@@ -0,0 +1,6083 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should probably delete this file!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good catch!!

beckykd
beckykd previously approved these changes Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Combine QPU information and Get backend information qpu-information and get-qpu-information

5 participants