Skip to content

Transpilation page fix#4814

Merged
abbycross merged 22 commits intomainfrom
ajc/de-tutorialize
Mar 19, 2026
Merged

Transpilation page fix#4814
abbycross merged 22 commits intomainfrom
ajc/de-tutorialize

Conversation

@abbycross
Copy link
Copy Markdown
Collaborator

Closes #4727.

@jyu00 does this help the reading experience?

@abbycross abbycross requested a review from a team March 13, 2026 18:29
@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
Collaborator

@jyu00 jyu00 left a comment

Choose a reason for hiding this comment

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

Since this page is about comparing the settings, it really should be moved to the end of the Transpilation section. Otherwise it's odd, for example, to talk about setting optimization level prior to comparison the result of different levels.

Comment thread docs/guides/circuit-transpilation-settings.ipynb Outdated
Comment thread docs/guides/circuit-transpilation-settings.ipynb Outdated
Comment thread docs/guides/circuit-transpilation-settings.ipynb
@abbycross
Copy link
Copy Markdown
Collaborator Author

Ready for final review @jyu00 - thanks!

jyu00
jyu00 previously approved these changes Mar 18, 2026
Copy link
Copy Markdown
Collaborator

@jyu00 jyu00 left a comment

Choose a reason for hiding this comment

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

Some comments about the code, but the rest looks good!

Comment thread docs/guides/circuit-transpilation-settings.ipynb
Comment thread docs/guides/circuit-transpilation-settings.ipynb Outdated
Comment thread docs/guides/circuit-transpilation-settings.ipynb Outdated
Co-authored-by: Jessie Yu <jessieyu@us.ibm.com>
Copy link
Copy Markdown
Collaborator

@jyu00 jyu00 left a comment

Choose a reason for hiding this comment

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

Thanks Abby!

Copy link
Copy Markdown
Collaborator

@kaelynj kaelynj left a comment

Choose a reason for hiding this comment

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

🚀

@abbycross abbycross added this pull request to the merge queue Mar 19, 2026
"---\n",
"title: Compare transpiler settings\n",
"description: In this tutorial, we'll explore the transpilation pipeline and take you through the full process of creating, transpiling, and submitting circuits.\n",
"description: explore the transpilation pipeline through the full process of creating, transpiling, and submitting circuits.\n",
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"description: explore the transpilation pipeline through the full process of creating, transpiling, and submitting circuits.\n",
"description: Explore the transpilation pipeline through the full process of creating, transpiling, and submitting circuits.\n",

Merged via the queue into main with commit e673286 Mar 19, 2026
5 checks passed
@abbycross abbycross deleted the ajc/de-tutorialize branch March 19, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Out of place transpilation page

6 participants