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

Upgrade PyO3 and rust-numpy to 0.21.x and Bound API #1152

Merged
merged 4 commits into from
Apr 2, 2024

Conversation

mtreinish
Copy link
Member

@mtreinish mtreinish commented Apr 1, 2024

This commit updates rustworkx to the latest release 0.21.x and updates the API usage for the new Bound api. [1] Luckily in rustworkx the usage of py references was minimal so the migration effort wasn't too complex, it's mostly the iterators.rs module's custom traits that needed to updated to handle the new API.

Fixes #1150

[1] https://pyo3.rs/v0.21.0/migration#from-020-to-021

This commit updates rustworkx to the latest release 0.21.0 and updates
the API usage for the new Bound api. [1] Luckily in rustworkx the usage
of py references was minimal so the migration effort wasn't too complex,
it's mostly the iterators.rs module's custom traits that needed to updated
to handle the new API.

Fixes Qiskit#1150

[1] https://pyo3.rs/v0.21.0/migration#from-020-to-021
@coveralls
Copy link

coveralls commented Apr 1, 2024

Pull Request Test Coverage Report for Build 8523368212

Details

  • 123 of 129 (95.35%) changed or added relevant lines in 13 files are covered.
  • 6 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.02%) to 96.553%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/digraph.rs 15 17 88.24%
src/graph.rs 13 15 86.67%
src/iterators.rs 36 38 94.74%
Files with Coverage Reduction New Missed Lines %
src/shortest_path/all_pairs_bellman_ford.rs 6 95.53%
Totals Coverage Status
Change from base Build 8516612768: 0.02%
Covered Lines: 17142
Relevant Lines: 17754

💛 - Coveralls

Cargo.lock Outdated Show resolved Hide resolved
@mtreinish mtreinish changed the title Upgrade PyO3 and rust-numpy to 0.21.0 and Bound API Upgrade PyO3 and rust-numpy to 0.21.x and Bound API Apr 1, 2024
@IvanIsCoding IvanIsCoding added the automerge Queue a approved PR for merging label Apr 2, 2024
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.

I am glad the transition was smoother than I expected. Thanks for updating!

@mergify mergify bot merged commit 9f0646e into Qiskit:main Apr 2, 2024
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Queue a approved PR for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update PyO3 to version 0.21.x
4 participants