Skip to content

Conversation

@jlapeyre
Copy link
Contributor

skip_transpilation no longer exists. The main point of the preceding step was translating to the target backend. There is no need to comment further on the relation between these steps.

Closes #4266

`skip_transpilation` no longer exists. The main point of the preceding
step was translating to the target backend. There is no need to comment
further on the relation between these steps.
@qiskit-bot
Copy link
Contributor

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

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

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! I noticed this same line also appeared in a couple of other notebooks, even though they don't use the skip_transpilation option so I've pushed a commit removing them there as well. Let me know if you think that's appropriate.

Copy link
Collaborator

@miamico miamico left a comment

Choose a reason for hiding this comment

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

Great, thanks for taking care of this @jlapeyre ! LGTM

@jlapeyre
Copy link
Contributor Author

LGTM. I am good with merging it.

@frankharkins frankharkins added this pull request to the merge queue Nov 25, 2025
Merged via the queue into Qiskit:main with commit e4e8450 Nov 25, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Tutorial mentions non existent option

4 participants