Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Vendor Qiskit's pubsub implementation #799

Merged
merged 2 commits into from
Jan 22, 2024
Merged

Conversation

jakelishman
Copy link
Member

Summary

Qiskit 1.0 is removing this, so applications that want to continue using it (within themselves; Qiskit will no longer be a central broker) should arrange for a centralised location for their published events. This vendors the code into the IBM Provider, from which Runtime can then re-import.

For backwards compatibility, the IBM Provider's _Broker singleton will resolve to the Qiskit one, if a version of Qiskit that supplies it is installed. This means that the package remains compatible with code that still accesses the Subscriber via Qiskit (e.g. an un-upgraded IBM Runtime).

Details and comments

Qiskit 1.0 is removing this, so applications that want to continue using
it (within themselves; Qiskit will no longer be a central broker) should
arrange for a centralised location for their published events.  This
vendors the code into the IBM Provider, from which Runtime can then
re-import.

For backwards compatibility, the IBM Provider's `_Broker` singleton will
resolve to the Qiskit one, if a version of Qiskit that supplies it is
installed.  This means that the package remains compatible with code
that still accesses the `Subscriber` via Qiskit (e.g. an un-upgraded IBM
Runtime).
@coveralls
Copy link

Pull Request Test Coverage Report for Build 7613694488

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.03%) to 51.534%

Totals Coverage Status
Change from base Build 7572851713: 0.03%
Covered Lines: 3476
Relevant Lines: 6745

💛 - Coveralls

@kt474 kt474 merged commit 052c490 into Qiskit:main Jan 22, 2024
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants