Skip to content

Use new API URL - #2192

Closed
Kevin Tian (kt474) wants to merge 3 commits into
Qiskit:mainfrom
kt474:test-new-iqp-api
Closed

Kevin Tian (kt474) wants to merge 3 commits into
Qiskit:mainfrom
kt474:test-new-iqp-api

Conversation

@kt474

@kt474 Kevin Tian (kt474) commented Mar 21, 2025 •

Copy link
Copy Markdown
Contributor

Summary

Use the new API URLs

staging: https://quantum.test.cloud.ibm.com/api
production: https://quantum.cloud.ibm.com/api

Details and comments

Fixes #2147

@kt474 Kevin Tian (kt474) changed the title Testing new IQP API Use new API URL Apr 10, 2025
@kt474
Kevin Tian (kt474) marked this pull request as ready for review April 10, 2025 18:33

@ElePT Elena Peña Tapia (ElePT) left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks Kevin Tian (@kt474), it looks like the new URL is already active and working, right? do you see any blocker for this PR or should we go ahead and merge it?

From the release note perspective, I am debating whether we should add an API Changes release note or not. It seems reasonable to document the change, but it's not the definition of "user facing" that I am used to.

@ElePT Elena Peña Tapia (ElePT) added this to the 0.39.0 milestone Apr 22, 2025
@kt474

Copy link
Copy Markdown
Contributor Author

Thanks Kevin Tian (@kt474), it looks like the new URL is already active and working, right? do you see any blocker for this PR or should we go ahead and merge it?

I'll follow up with the IQP side, I think we'll have to do more testing with the the URLs

From the release note perspective, I am debating whether we should add an API Changes release note or not. It seems reasonable to document the change, but it's not the definition of "user facing" that I am used to.

We can document any discrepancies between the APIs that effect the user. But if they're exactly the same then there's no need.

@ElePT

Copy link
Copy Markdown
Collaborator

This PR has been superseded by #2239

@kt474
Kevin Tian (kt474) deleted the test-new-iqp-api branch May 28, 2025 13:47
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.

New IQP API integration

2 participants