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

Support control flow in LookaheadSwap #9429

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

Support control flow in LookaheadSwap #9429

jakelishman opened this issue Jan 23, 2023 · 0 comments
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 LookaheadSwap 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 lookahead is a barely-used version of routing, and we're fairly sure it has latent bugs even in non-control-flow contexts.

@jakelishman jakelishman added type: feature request New feature or request priority: low mod: transpiler Issues and PRs related to Transpiler labels Jan 23, 2023
@jakelishman jakelishman changed the title Support control-flow in LookaheadSwap Support control flow in LookaheadSwap Jan 23, 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 priority: low type: feature request New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant