Client does not hang

[marcorossi5]

This PR adds a wait_for_results boolean flag to Client.run_circuit method to let the user choose if the client should hang or not waiting for the server returning the results.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (2e358ef) to head (cfbdb01).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main       #42   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         7    +2     
  Lines          117       199   +82     
=========================================
+ Hits           117       199   +82     

Flag	Coverage Δ
unittests	100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[scarrazza]

This is missing a function which downloads the results from the server:

r = client.get_results(pid=''')

[marcorossi5]

This new commit implements the Qiskit-like interface of QiboClient and tests.

Address #22, #41, #46, #49

[scarrazza]

@marcorossi5 could you please include the following changes too:

• update the example in https://github.com/qiboteam/qibo-client/blob/feature/client-does-not-hang/examples/run_error_job.py
• update the README file with the new workflow (job, .result())

[marcorossi5]

@scarrazza, coverage is 100% now. Can you merge?

[scarrazza]

@marcorossi5, do we have a client.get_job(jobid) or similar function?

[marcorossi5]

Not yet. It can be useful. Let me implement it in a separate branch after we merge these two if you agree

[scarrazza]

ok, thanks.

[scarrazza]

Do you really need this function outside the QiboJob class? If you move there as a private function _wait_for_response... you can access the status and avoid all the problems raised in the discussion above.