Skip to content

change backend name to Torino #2766

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

chongyouquan
Copy link

@chongyouquan chongyouquan commented Mar 11, 2025

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@qiskit-bot
Copy link
Contributor

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:

Copy link
Member

@frankharkins frankharkins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! This approach should work fine. However, I don't think we have access to Fez from our testing instance, can we use ibm_torino instead?

Then it's probably easiest if I run the notebook and push to this branch.

@chongyouquan
Copy link
Author

chongyouquan commented Mar 12, 2025

Thanks! This approach should work fine. However, I don't think we have access to Fez from our testing instance, can we use ibm_torino instead?

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 chongyouquan changed the title change backend name to Fez change backend name to Torino Mar 12, 2025
@frankharkins
Copy link
Member

@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.

@eginez
Copy link

eginez commented Mar 13, 2025

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

@frankharkins
Copy link
Member

frankharkins commented Mar 14, 2025

@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 (73eecd3), please let me know if this is ok:

  1. I've added a version-info cell 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.
  2. 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.
  3. 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 Roll back sampler results #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: {}

@eginez
Copy link

eginez commented Mar 14, 2025

@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 #2775

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants