Remove IQP Classic #2289
Remove IQP Classic #2289
Conversation
|
I think this PR is ready for an initial review. Just like the Qiskit 2.0 support PR/release we may have to remove some methods without deprecation since they are no longer supported on the server anyway. Update: on hold for at least another month |
There was a problem hiding this comment.
Thanks a lot for doing this @kt474, removals are never easy, but it is going to be sooo nice to see the cleaner logic once the channel is gone. I went through the code changes without looking at the tests yet and left a few questions and comments. Overall, I think that the PR is in a good place, but we'll have to use this extra time we get before the merging date to test the service and make sure we are not leaving any loose ends. I will take a look at the tests in a follow-up.
|
I think this is in a mergable state, we can keep a close eye on any followup |
Summary
Things I noticed that are handled in a separate PR (#2295).
TODO before merging:
ibm_quantumaccounts saved doesn't break anythingDetails and comments
Fixes #2285