Skip to content
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

Use qiskit.qpy.QPY_VERSION (up to 13) instead of always QPY 11 #2096

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

1ucian0
Copy link
Member

@1ucian0 1ucian0 commented Jan 7, 2025

Summary

This PR always use the Qiskit supported QPY version or 13, if Qiskit SDK is ahead of server. Before merging this, server side needs to confirm that QPY 13 is available.

Details and comments

Fixes #2095

Additionally, this PR increases the minimum version for qiskit to 1.3, making unnecessary the symengine and _TERRA_VERSION branches.

on hold because the following blockers:

@1ucian0 1ucian0 added the on hold label Jan 7, 2025
@1ucian0 1ucian0 requested a review from jyu00 January 8, 2025 08:54
- |
The minimal ``qiskit`` version is now 1.3.
- |
The QPY payload now is generated based on ``qiskit.qpy.QPY_VERSION`` and capped until version 13.
Copy link
Member

Choose a reason for hiding this comment

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

We switched to using towncrier for our release notes - https://github.com/Qiskit/qiskit-ibm-runtime/blob/main/CONTRIBUTING.md#adding-a-new-release-note. You can add a release note to the /unreleased folder with a file name of 2096.upgrade.rst.

Copy link
Member Author

Choose a reason for hiding this comment

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

oops, fixed in fbdc01e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use QPY version 13
2 participants