change backend name to Torino
Check out this pull request on ![]()
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
Thanks for contributing to Qiskit documentation!
Before your PR can be merged, it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. Thanks! 🙌
One or more of the following people are relevant to this code:
- @ThomasMerkh
- @abbycross
- @alexshih
- @dcanuto
- @johannesgreiner
- @jrowenwu
- @kaelynj
- @pandasa123
Thanks! This approach should work fine. However, I don't think we have access to Fez from our testing instance, can we use
ibm_torinoinstead?Then it's probably easiest if I run the notebook and push to this branch.
@frankharkins: Ok done, changed the backend name to Torino. Could you take a look again? Thanks!
Also, is it possible to manually trigger a re-run of this notebook with the change?
@chongyouquan sorry for the delay, I'm trying to run the notebook but am getting an error with Qiskit Serverless that I'm struggling to debug. If you want to try it yourself, you can run this command from the root of this repo, then commit the results and push.
tox exec -- test-docs-notebooks docs/guides/q-ctrl-performance-management.ipynb --write
I'll continue looking into the serverless error.
Hi @frankharkins thanks for looking into the problem, is there a chance you can share the stacktrace of the problem? We identified an issue with qiskit 1.4 recently. Also try the below dependencies to test that notebook
❯ cat requirements.txt|grep qiskit
qiskit==1.2.4
qiskit-ibm-catalog==0.2.0
qiskit-ibm-runtime==0.33.2
qiskit-serverless==0.18.1
@eginez thanks, I'm running now with those requirements but it's taking quite a long time. I'll push the results as soon as it's done. In the meantime, I've pushed a couple of small changes to the notebook (73eecd3c9372da404dfe9515ff30e4e1bfe903c9), please let me know if this is ok:
- I've added a
version-infocell to the notebook: This is just an empty cell for now, but when we run the notebook with our script, it'll populate with the package versions used so users (and ourselves) can reproduce the results in the future. All our notebooks should have one of these, we just overlooked it when adding this guide. - Rather than printing the full output of
get_counts(), I've changed it to print the first 10 items. This is just because the full counts are super long. - Finally, I've added a hidden cell that asserts the hidden bitstring is the most frequent result. This means the notebook execution will fail if the device results are bad, which will stop us committing them. This should prevent the issue described in #2775.
Also, here's the stack trace, although it's not very helpful:
Stack trace
---------------------------------------------------------------------------
QiskitServerlessException Traceback (most recent call last)
Cell In[8], line 2
1 # Retrieve the counts from the result list
----> 2 result = qctrl_estimator_job.result()
File ~/Documents/documentation/.venv/lib/python3.11/site-packages/qiskit_serverless/core/job.py:190, in Job.result(self, wait, cadence, verbose, maxwait)
187 results = self._job_service.result(self.job_id)
189 if self.status() == "ERROR":
--> 190 raise QiskitServerlessException(results)
192 if isinstance(results, str):
193 try:
QiskitServerlessException: {}
@frankharkins Excellent thanks for that, those changes look good.
On the stacktrace, nop that looks different. I was observing an issue with qpy serialization. And I hear on the execution, it took me some time to get the circuit executed, but I was able to confirm that once we use the right devices, this problem gets fixed and the content is basically the same a https://github.com/Qiskit/documentation/pull/2775