Skip to content
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

Restore level 1 layout/routing default to SabreSwap #9422

Closed
Tracked by #9417
jakelishman opened this issue Jan 23, 2023 · 0 comments · Fixed by #10371
Closed
Tracked by #9417

Restore level 1 layout/routing default to SabreSwap #9422

jakelishman opened this issue Jan 23, 2023 · 0 comments · Fixed by #10371
Labels
mod: transpiler Issues and PRs related to Transpiler type: feature request New feature or request
Milestone

Comments

@jakelishman
Copy link
Member

What should we add?

Part of #9417. Depends on:

In #8830 we had to revert optimisation level 1 to use its legacy behaviour of laying out with DenseLayout and routing with StochasticSwap if the circuit was a control-flow circuit, since the new default SabreSwap couldn't handle the control-flow. Once SabreSwap supports control-flow, we should remove this switching logic, and swap back to the preferred router.

SabreLayout should be easier to upgrade to initial support of control-flow (especially since it'll technically already run, it'll just not have good output), if only the Python-space paths are upgraded.

@jakelishman jakelishman added the type: feature request New feature or request label Jan 23, 2023
@jakelishman jakelishman added this to the 0.24.0 milestone Jan 23, 2023
@jakelishman jakelishman added the mod: transpiler Issues and PRs related to Transpiler label Jan 23, 2023
@jakelishman jakelishman modified the milestones: 0.24.0, 0.25.0 Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mod: transpiler Issues and PRs related to Transpiler type: feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant