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

setup.py requirements for qiskit are outdated #2096

Closed
woodsp-ibm opened this issue Apr 4, 2024 · 2 comments
Closed

setup.py requirements for qiskit are outdated #2096

woodsp-ibm opened this issue Apr 4, 2024 · 2 comments

Comments

@woodsp-ibm
Copy link
Member

woodsp-ibm commented Apr 4, 2024

What is the current behavior?

https://github.com/Qiskit/qiskit-aer/blob/main/setup.py#L24-L28

As V2 primitives are now included this should be made dependent on Qiskit 1.0 or above I believe.

I imagine latest Aer being installed with an older Qiskit is the cause of the failure reported here https://qiskit.slack.com/archives/C7SS31917/p1712246299460849

@jakelishman
Copy link
Member

If all that Aer 0.14 needs from Qiskit 1.0 is the V2 primitives, it would be good to only conditionally expose the V2 primitives if Qiskit is 1.0+, and leave Aer 0.14 compatible with Qiskit pre-1.0, since we still officially support that.

@doichanj
Copy link
Collaborator

doichanj commented Apr 5, 2024

I put conditional import of Primitives V2 in PR #2094

@doichanj doichanj closed this as completed May 1, 2024
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

No branches or pull requests

3 participants