Skip to content

Use default calibration and refine the logical-noise PEC convergence plot - #5629

Merged
abbycross merged 2 commits into
mainfrom
follow-up-logical-noise-pec
Sep 10, 2026
Merged

Use default calibration and refine the logical-noise PEC convergence plot#5629
abbycross merged 2 commits into
mainfrom
follow-up-logical-noise-pec

Conversation

@henryzou50

@henryzou50 Henry Zou (henryzou50) commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

This follow-up to #5616 removes the custom calibration_id and its outdated explanatory note so the tutorial uses the backend’s default calibration. It also narrows the convergence plot’s y-axis limits from ±0.18 to ±0.08, as Caleb suggested.

Current status

  • All eight repository quality checks passed.
  • The notebook still requires xslow; default-calibration execution has not yet been validated.
  • The plotting code is updated, but the saved convergence chart still uses the previous limits.

Before merging

  • Confirm xslow is deployed on ibm_boston.
  • Verify xslow executes using the default calibration without a custom calibration_id. Small hardware test passed per Caleb’s recommendation, a full experiment rerun is not required.
  • Keep the plotting code consistent with the saved chart. The axis-limit adjustment is deferred until the chart can be regenerated from saved results.
  • Obtain final review.

Remove the custom calibration_id and outdated calibration note.
Narrow the convergence plot y-axis limits from ±0.18 to ±0.08.
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@qiskit-bot

Copy link
Copy Markdown
Contributor

One or more of the following people are relevant to this code:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@abbycross
abbycross added this pull request to the merge queue Sep 10, 2026
Merged via the queue into main with commit 35f4df8 Sep 10, 2026
5 checks passed
@abbycross
abbycross deleted the follow-up-logical-noise-pec branch September 10, 2026 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants