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

Update UnitaryGate import now that qiskit.extensions is deprecated #463

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

garrison
Copy link
Member

@garrison garrison commented Dec 7, 2023

qiskit.extensions is deprecated starting with Qiskit 0.45, and the UnitaryGate class has moved to qiskit.circuit.library. However, this class did not exist at this location prior to Qiskit 0.45, so this means having to bump the minimum Qiskit version as well.

@garrison garrison added the dependencies Pull requests that update a dependency file label Dec 7, 2023
@coveralls
Copy link

coveralls commented Dec 7, 2023

Pull Request Test Coverage Report for Build 7202329089

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 92.498%

Totals Coverage Status
Change from base Build 7189163548: 0.0%
Covered Lines: 2984
Relevant Lines: 3226

💛 - Coveralls

@garrison garrison marked this pull request as ready for review December 8, 2023 16:48
@garrison
Copy link
Member Author

If we merge this before the next release, it will allow us to knock out a few DeprecationWarnings at the cost of requiring a more recent qiskit (i.e., one released in November rather than August). I don't think this is a huge down side, but there's also no urgency to merge this before the next release. (We'll definitely have to merge it before CKT can work with Qiskit 1.0, however.)

@caleb-johnson caleb-johnson merged commit 459ee0d into main Feb 7, 2024
11 checks passed
@caleb-johnson caleb-johnson deleted the unitarygate-moved branch February 7, 2024 16:38
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants