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

Bump PyO3 and rust-numpy to 0.23 #1364

Merged
merged 3 commits into from
Jan 16, 2025
Merged

Conversation

mtreinish
Copy link
Member

@mtreinish mtreinish commented Jan 15, 2025

Closes #1325

This commit bumps the version of pyo3 and rust-numpy used by qiskit to the latest release 0.23. The largest change by volume of code is the deprecation of all the *_bound() methods. These are just warnings but they would be fatal to our CI so it needs to be updated. The larger functional change that required updating the code is the change in the traits around converting to Python objects.

As a side effect of this change it lets us unify the hashbrown versions installed because we can update indexmap.

This commit bumps the version of pyo3 and rust-numpy used by qiskit to
the latest release 0.23. The largest change by volume of code is the
deprecation of all the *_bound() methods. These are just warnings but
they would be fatal to our CI so it needs to be updated. The larger
functional change that required updating the code is the change in the
traits around converting to Python objects.

As a side effect of this change it lets us unify the hashbrown versions
installed because we can update indexmap.
@mtreinish mtreinish added this to the 0.16.0 milestone Jan 15, 2025
@IvanIsCoding
Copy link
Collaborator

I need to read this more carefully but first: thanks for working on this!

It was one of those updates with a lot of breaking changes I am glad we are able to migrate without too many headaches.

@coveralls
Copy link

coveralls commented Jan 15, 2025

Pull Request Test Coverage Report for Build 12815647537

Details

  • 274 of 282 (97.16%) changed or added relevant lines in 17 files are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.02%) to 95.811%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/json/mod.rs 15 16 93.75%
src/toposort.rs 3 5 60.0%
src/generators.rs 8 13 61.54%
Files with Coverage Reduction New Missed Lines %
src/digraph.rs 2 98.29%
Totals Coverage Status
Change from base Build 12797615868: -0.02%
Covered Lines: 18343
Relevant Lines: 19145

💛 - Coveralls

Copy link
Collaborator

@IvanIsCoding IvanIsCoding left a comment

Choose a reason for hiding this comment

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

LGTM. Some of the new lifetimes with ’py and the new methods possibly returning errors make the code a little more annoying but it is what it is

Cargo.toml Outdated Show resolved Hide resolved
@IvanIsCoding IvanIsCoding added this pull request to the merge queue Jan 16, 2025
Merged via the queue into Qiskit:main with commit c05a583 Jan 16, 2025
31 checks passed
@mtreinish mtreinish deleted the use-pyo3-23 branch January 24, 2025 04:01
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.

Update to PyO3 0.23
3 participants