CQCL / pytket-iqm

pytket-iqm, extensions for pytket quantum SDK
Apache License 2.0
2 stars 4 forks source link

Update to iqm-client 4.2 and replace settings file #6

Closed cqc-alec closed 2 years ago

cqc-alec commented 2 years ago

I tried re-enabling the remote tests but they are still failing, now with errors like:

>       res = b.run_circuit(c, n_shots=n_shots, timeout=30)

backend_test.py:49: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
../env/lib/python3.9/site-packages/pytket/backends/backend.py:384: in run_circuit
    return self.run_circuits(
../env/lib/python3.9/site-packages/pytket/backends/backend.py:409: in run_circuits
    results = self.get_results(handles, **kwargs)
../env/lib/python3.9/site-packages/pytket/backends/backend.py:352: in get_results
    return [self.get_result(handle, **kwargs) for handle in handles]
../env/lib/python3.9/site-packages/pytket/backends/backend.py:352: in <listcomp>
    return [self.get_result(handle, **kwargs) for handle in handles]
../pytket/extensions/iqm/backends/iqm.py:294: in get_result
    self._client.wait_for_results(run_id, timeout_secs=timeout)
../env/lib/python3.9/site-packages/iqm_client/iqm_client.py:570: in wait_for_results
    results = self.get_run(job_id)
../env/lib/python3.9/site-packages/iqm_client/iqm_client.py:522: in get_run
    result = RunResult.from_dict(result.json())
../env/lib/python3.9/site-packages/iqm_client/iqm_client.py:268: in from_dict
    return RunResult(status=Status(input_copy.pop('status')), **input_copy)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

>   ???
E   pydantic.error_wrappers.ValidationError: 1 validation error for RunResult
E   metadata
E     field required (type=value_error.missing)

pydantic/main.py:341: ValidationError
cqc-alec commented 2 years ago

From my side this can be merged now, not sure if we should wait for external feedback?

OK thanks, will merge this. Keeping #2 open.

cqc-melf commented 2 years ago

I still would wait to release this until the remote checks are working again?