-
Notifications
You must be signed in to change notification settings - Fork 127
Add Qiskit Runtime 0.18 docs #729
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly LGTM, good job figuring out the provider
weirdness!
I noticed api/qiskit-ibm-runtime/qiskit_ibm_provider.transpiler.passes.basis
was empty, should it include ConvertIdToDelay
? (I don't fully understand sphinx 😢)
Good point. I asked Kevin Tian over DM to clarify. Either way, we should fix that in a follow up since we're behind on getting these docs out. |
The 0.17 docs are moved to be historical via `npm run make-historical -- -p qiskit-ibm-runtime`. The 0.18 docs were generated with `npm run gen-api -- -p qiskit-ibm-runtime -v 0.18.0 -a https://github.com/Qiskit/qiskit-ibm-runtime/actions/runs/7703740124/artifacts/1204347771`. Note that they include two new top-level modules, closing Qiskit#238. This PR resulted in discovering Qiskit#732. This has always been a problem we didn't know about. I manually ignore the broken links for now to unblock the Runtime 0.18 release and to close Qiskit#238.
The 0.17 docs are moved to be historical via
npm run make-historical -- -p qiskit-ibm-runtime
.The 0.18 docs were generated with
npm run gen-api -- -p qiskit-ibm-runtime -v 0.18.0 -a https://github.com/Qiskit/qiskit-ibm-runtime/actions/runs/7703740124/artifacts/1204347771
. Note that they include two new top-level modules, closing #238.This PR resulted in discovering #732. This has always been a problem we didn't know about. I manually ignore the broken links for now to unblock the Runtime 0.18 release and to close #238.