Skip to content

Repetition codes experiment extension#5143

Draft
rdputteribm wants to merge 2 commits into
Qiskit:mainfrom
rdputteribm:repetition-codes-experiment-extension
Draft

Repetition codes experiment extension#5143
rdputteribm wants to merge 2 commits into
Qiskit:mainfrom
rdputteribm:repetition-codes-experiment-extension

Conversation

@rdputteribm
Copy link
Copy Markdown

Changed the tutorial into a quantum memory experiment with multiple cycles of error correction, as discussed with @nathanearnestnoble .

Also brought the tutorial (more) in line with the tutorial template.

@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

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 20, 2026

CLA assistant check
All committers have signed the CLA.

Comment thread docs/tutorials/repetition-codes.ipynb Outdated
Comment thread docs/tutorials/repetition-codes.ipynb Outdated
Comment thread docs/tutorials/repetition-codes.ipynb Outdated
Copy link
Copy Markdown
Collaborator

@henryzou50 henryzou50 left a comment

Choose a reason for hiding this comment

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

Overall this tutorial update looks great, and thanks for the changes! A few notes:

Structure vs. template: Having only a large-scale hardware example here is fine, but per the template, I believe should still include both the ## Small-scale simulator example and ## Large-scale hardware example headers, with a brief explanation under the small-scale section about why we're skipping the simulator example in this case, similar to how we did it for other tutorials.

Tutorial Survey: The link is correct, but most of the other updated tutorials don't include this section. Probably fine to keep, just flagging for consistency.

I pushed some minor changes for grammar, typos, and consistency, but otherwise great job!

@rdputteribm
Copy link
Copy Markdown
Author

Hi @henryzou50 ,
Thanks a lot for your review!

Structure vs. template: Having only a large-scale hardware example here is fine, but per the template, I believe should still include both the ## Small-scale simulator example and ## Large-scale hardware example headers, with a brief explanation under the small-scale section about why we're skipping the simulator example in this case, similar to how we did it for other tutorials.

I am happy to add an empty simulator header + explanation if that is preferred (I realize it's inthe template), but would have a slight preference for keeping it simple and just keep leaving it out. What is an example of a tutorial that does not do a small-scale simulator run, but still has a header for it?

Tutorial Survey: The link is correct, but most of the other updated tutorials don't include this section. Probably fine to keep, just flagging for consistency.

Good catch. Since the survey is not in the template, let me just remove it.

…tured tutorial to match template format

- Added Learning outcomes section with 3 key outcomes
- Added Prerequisites section with platform content references
- Changed Step 1-4 headers to subsections (###)
- Added job tagging (TUT-RC) for hardware execution
- Added Next steps section with recommendations

further cleanup

further cleanup

final updates of original tutorial workflow

implemented logical memory experiment for repetition code

added more documentation/explanation

update to text

little fixes for IBM Style

./fix

spelling

fix links

tox -e fix  + some typo fixes

Fix the lint issues and some minor grammar and typo fixes.

Update . to : for step headers

removed survey
@rdputteribm rdputteribm force-pushed the repetition-codes-experiment-extension branch from 355b29b to a03a634 Compare June 2, 2026 13:35
@henryzou50 henryzou50 force-pushed the repetition-codes-experiment-extension branch from 9baa54d to a03a634 Compare June 2, 2026 19:00
@henryzou50
Copy link
Copy Markdown
Collaborator

What is an example of a tutorial that does not do a small-scale simulator run, but still has a header for it?

@rdputteribm Good question, there's a clean example for this. combine-error-mitigation-techniques keeps the ## Small-scale simulator example header but skips the actual run, with just a one-line explanation of why:

Small-scale simulator example

We will forgo this step since runtime error mitigation is not supported on simulators.

It then continues straight into the ## Large-scale hardware example section.

Given that, I'd lean toward following the template here: keep both the ## Small-scale simulator example and ## Large-scale hardware example headers, and add a brief note under the small-scale one explaining why we skip the simulator in this case. It keeps us consistent with the template and the other tutorials while staying minimal.

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