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 qiskit-aqt-provider to 1.8.x #113

Merged
merged 2 commits into from
Aug 5, 2024

Conversation

jbrixon
Copy link
Contributor

@jbrixon jbrixon commented Aug 5, 2024

Description

Updates the Qiskit AQT Provider to the latest version, so that it works better with the latest version of the API and benefits from a few other new developments.

This pins the dependency to the minor version, which will remain backwards compatible. The Qiskit provider doesn't strictly follow semver.

Related issues

N/A

Checklist

  • I have performed a self-review of my code.
  • I have commented hard-to-understand parts of my code.
  • I have made corresponding changes to the public API documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have updated the changelog with any user-facing changes.

@jbrixon jbrixon requested a review from cqc-melf as a code owner August 5, 2024 10:38
Copy link
Collaborator

@cqc-melf cqc-melf left a comment

Choose a reason for hiding this comment

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

Looks good, can you maybe add something to the changelog?

@jbrixon
Copy link
Contributor Author

jbrixon commented Aug 5, 2024

Sure thing!

@fg-aqt
Copy link

fg-aqt commented Aug 5, 2024

@cqc-melf would it be possible to create a release after this merge (or once the integration with the AQT API is complete, if that is not the case yet)?

@cqc-melf
Copy link
Collaborator

cqc-melf commented Aug 5, 2024

@cqc-melf would it be possible to create a release after this merge (or once the integration with the AQT API is complete, if that is not the case yet)?

Yes, I will check if there is anything more we want to include and then do the release

@trvto
Copy link
Collaborator

trvto commented Aug 5, 2024

@cqc-melf I'm not ready to release my new routing routines yet. So happy to release the current state

@cqc-melf cqc-melf merged commit 4369e21 into CQCL:main Aug 5, 2024
20 checks passed
@cqc-melf
Copy link
Collaborator

cqc-melf commented Aug 6, 2024

@jbrixon @fg-aqt the release should be out now. If you have any issues with it, please let me know

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.

4 participants