-
Notifications
You must be signed in to change notification settings - Fork 88
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
Add "Save and retrieve Qiskit objects" page #1099
Conversation
…dd/save-and-retrieve
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
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.
Reads very well!! Couple of tweaks in my suggestions, but approving overall.
Co-authored-by: abbycross <across@us.ibm.com>
…dd/save-and-retrieve
It may be worth noting that QPY only provides backward compatibility but not forward. So it may not work if you are trying to use Qiskit version N-1 to load a payload dumped from version N. |
@jyu00 does this sentence describe it correctly?
|
Yes, but the goal here is to warn users they cannot load a QPY file using a down level Qiskit. So I think the sentence would better worded as QPY being backward compatible but not forward compatible (since it cannot predict the future). Also Qiskit 1.0 added versions to QPY. So one can look up the version of a QPY file and compare that to the QPY versions support by their Qiskit. @mtreinish can give you more details if needed. |
I think the confusion comes from the file type vs. the QPY program. Does this remove the ambiguity?
It removes mention of forward/backwards compatibility and makes it clearer that older versions of Qiskit might not understand the file. EDIT: Approved by Jessie via Slack |
Continuation of Qiskit#956 [Link to preview](https://qiskit-docs-preview-pr-1099.1799mxdls7qz.us-south.codeengine.appdomain.cloud/analyze/save-and-retrieve). --------- Co-authored-by: nick bronn <ntbronn@us.ibm.com> Co-authored-by: abbycross <across@us.ibm.com>
PRs Qiskit#977 and Qiskit#1099 added some new pages in anticipation of a restructure which has since been postponed. This PR fits those pages into our current content structure.
Continuation of #956
Link to preview.