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.

Sign up for GitHub

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

Jump to bottom

Use polling for job status instead of websockets #1778

Merged

kt474 merged 2 commits into Qiskit:main from kt474:remove-websockets

Jul 1, 2024

Member

kt474 commented Jun 27, 2024

Summary

Can fully remove the websocket code after the deprecation period in #1776

Details and comments

Fixes #1775


          Use polling for job status instead of websockets

b970723

coveralls commented Jun 27, 2024 •

edited

Loading

Pull Request Test Coverage Report for Build 9702544921

Details

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

Totals
Change from base Build 9701107649:	0.06%
Covered Lines:	6109
Relevant Lines:	7437

💛 - Coveralls

ptristan3 approved these changes

View reviewed changes

kt474 mentioned this pull request

Update runtime_job_v2.py docs #1781

Closed


          Merge branch 'main' into remove-websockets

a790b25

coveralls commented Jul 1, 2024 •

edited

Loading

Pull Request Test Coverage Report for Build 9749510537

Details

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

Totals
Change from base Build 9749373790:	0.06%
Covered Lines:	6117
Relevant Lines:	7443

💛 - Coveralls

kt474 merged commit c607150 into Qiskit:main

18 checks passed

kt474 deleted the remove-websockets branch

July 1, 2024 19:04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet