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 for Qiskit 1.0 #50

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

frankharkins
Copy link

@frankharkins frankharkins commented Mar 4, 2024

Minor tweaks to make this project work with Qiskit 1.0

  • execute -> transpile & backend.run
  • qiskit.tools.visualization -> qiskit.visualization
  • qiskit.extensions.UnitaryGate -> qiskit.circuit.library.UnitaryGate

I couldn't get the test suite working on my machine but I can take a look if anything fails in CI.

@MarquessV
Copy link
Contributor

@frankharkins Thank you for doing this! I approved CI to run on this PR. Looks like tests are failing due to ImportError: cannot import name 'UnitaryGate' from 'qiskit.circuit.library'.

Also, are these changes intended to be compatible with the Qiskit version this library currently supports (>=0.38)? Just want to check if we should update the constraint in pyproject.toml.

@frankharkins
Copy link
Author

Hi, sorry I missed this. I've just pushed a commit that should add backwards compatibility for earlier Qiskit versions, so hopefully you won't need to update your dependencies.

@frankharkins
Copy link
Author

I'm not sure why the tests failed. QuantumCircuit.qasm has been removed so that will need updating, but that shouldn't have caused the test to fail as it's still using qiskit<=1.0. Unfortunately I can't look at it next week, but feel free to push to my branch or merge to a feature and work on it there. If it's still open I'll take a look when I'm back.

Here's the migration guide I've been using: https://docs.quantum.ibm.com/api/migration-guides/qiskit-1.0-features

The logs have expired
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.

2 participants