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.

Sign up for GitHub

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

Jump to bottom

Support control flow in BasicSwap #9430

Open
Tracked by #9417
jakelishman opened this issue Jan 23, 2023 · 0 comments · May be fixed by #8193
Open
Tracked by #9417

Support control flow in BasicSwap #9430

jakelishman opened this issue Jan 23, 2023 · 0 comments · May be fixed by #8193
Labels
mod: transpiler Issues and PRs related to Transpiler priority: low type: feature request New feature or request

Comments

@jakelishman
Copy link
Member

What should we add?

Low priority component of #9417.

Similar to #9419 for SabreSwap, it would be nice to bring all our available routing methods up-to-speed with control flow. See #9419 for more details on the current algorithm we have for routing control flow. Since BasicSwap is pure Python, its implementation would likely be very close to StochasticSwap's, and could hopefully re-use a lot of code.

This is low priority, since basic is a barely-used version of routing, and is almost entirely superseded by other methods.
@jakelishman jakelishman added type: feature request New feature or request mod: transpiler Issues and PRs related to Transpiler priority: low labels Jan 23, 2023
This was referenced Jan 23, 2023
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
mod: transpiler Issues and PRs related to Transpiler priority: low type: feature request New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add controlflow to basic_swap router ewinston/qiskit
1 participant
@jakelishman