Skip to content

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

Merged
merged 6 commits into from
Jan 31, 2024
Merged

Add Qiskit Runtime 0.18 docs #729

merged 6 commits into from
Jan 31, 2024

Conversation

Eric-Arellano
Copy link
Collaborator

@Eric-Arellano Eric-Arellano commented Jan 30, 2024

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.

@Eric-Arellano Eric-Arellano changed the title Add Qiskit Runtime 0.18 docs [wip] Add Qiskit Runtime 0.18 docs Jan 30, 2024
Copy link
Member

@frankharkins frankharkins left a 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 😢)

@Eric-Arellano Eric-Arellano changed the title [wip] Add Qiskit Runtime 0.18 docs Add Qiskit Runtime 0.18 docs Jan 31, 2024
@Eric-Arellano
Copy link
Collaborator Author

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.

@Eric-Arellano Eric-Arellano added this pull request to the merge queue Jan 31, 2024
Merged via the queue into main with commit 7439931 Jan 31, 2024
@Eric-Arellano Eric-Arellano deleted the EA/add-runtime-18 branch January 31, 2024 22:16
frankharkins pushed a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing API references for Runtime
2 participants