-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
JSONDecodeError
when callin run_circuit
#66
Comments
I confirm, @marcorossi5 could you please have a look asap? |
Yes, he is not using the latest qibo-client version. We have not released it yet ( |
I have just tried in PR #65 but it does not work too, same error. |
The former example was: import qibo
from qibo_client import Client
circuit = qibo.models.QFT(5)
token_path = Path(__file__).parent / "token.txt"
token = token_path.read_text()
client = Client(token)
job = client.run_circuit(circuit, nshots=100, device="sim")
r = job.result(verbose=True)
print(r.frequencies()) We should replace by this working example import qibo
from qibo_client import Client
circuit = qibo.Circuit(5)
circuit.add(qibo.gates.M(0))
token_path = Path(__file__).parent / "token.txt"
token = token_path.read_text()
client = Client(token)
job = client.run_circuit(circuit, nshots=100, device="sim")
r = job.result(verbose=True)
print(r.frequencies()) Any idea on why |
In principle we should be able to support circuit simulations without measurement gates (for sim partitions) and with measurement gates for hardware partitions. |
Ok, we can add some sort of check in the webapp for that. However, why is QFT circuit giving problems? Is it somehow deprecated or similar? |
As discussed, if you don't add the M gate, you should not expect samples/frequencies but just the state vector. |
I have just tested main and the latest release and for both setups I get the same failure. |
Some
qibo-cloud-backends
tests are failing due toqibo-client
qiboteam/qibo-cloud-backends#37.I am using
qibo-client 0.0.9
and running a circuit raisesJSONDecodeError: Expecting value: line 2 column 1 (char 1)
.The example in the README fails for instance.
The text was updated successfully, but these errors were encountered: