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

Consider updating symengine pin #1080

Open
bharat-thotakura opened this issue Oct 1, 2024 · 2 comments
Open

Consider updating symengine pin #1080

bharat-thotakura opened this issue Oct 1, 2024 · 2 comments
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request

Comments

@bharat-thotakura
Copy link
Contributor

bharat-thotakura commented Oct 1, 2024

What is happening?

We currently pin symengine~=0.11.0 after #874 in the requirements of qiskit-superstaq. However, that version seems to have issues building a wheel in a Python 3.13 environment, so we should pin the recently released v0.13.0 which installs as expected (and since qiskit>=1.0.0 supports symengine>=0.11).

Any additional context?

qiskit-superstaq version: 0.5.27
qiskit version: 1.2.2
Python version: 3.13.0rc2
Operating system: Ubuntu

@bharat-thotakura bharat-thotakura added enhancement New feature or request dependencies Pull requests that update a dependency file labels Oct 1, 2024
@bharat-thotakura
Copy link
Contributor Author

Related: Qiskit/qiskit#13251

@jakelishman
Copy link

Also Qiskit/qiskit#13259 - we borked the 1.2.3 release by being in too much of a rush, and yanked it. We'll make 1.2.4 shortly (tomorrow, most likely), which should be ok for any combination of symengine 0.11 and 0.13 between client and server, since ParameterExpression should limit our exposure to the full set of changes in symengine between those versions.

We hadn't come across the problem before, but it looks like you guys did in #874. We'll be working on reducing our dependence on symengine in the future. Near-term, we're hoping to cut it out of serialisation (Qiskit/qiskit#13252), and longer term, potentially remove the dependence entirely.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants