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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mtreinish
Copy link
Member

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

Pull Request Test Coverage Report for Build 12796359157

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 12794776080: -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

@@ -73,9 +73,6 @@ features = ["multi_thread"]

[features]
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can also delete the whole [features] section given we have none at the moment

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.

3 participants