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

[x/gamm][stableswap]: Update binary search upper bounds to reflect spec #3007

Closed
Tracked by #1451
AlpinYukseloglu opened this issue Oct 16, 2022 · 0 comments
Closed
Tracked by #1451

[x/gamm][stableswap]: Update binary search upper bounds to reflect spec #3007

AlpinYukseloglu opened this issue Oct 16, 2022 · 0 comments
Labels
C:stableswap C:x/gamm Changes, features and bugs related to the gamm module.

Comments

@AlpinYukseloglu
Copy link
Contributor

Background

We can reach even better estimates for our initial binary search upper bounds by following the linear approximation method describes in our spec.

Suggested Design

  • Update both two-asset and multi-asset binary search solvers to reflect upper bounds outlined in spec

Acceptance Criteria

  • All new and existing tests pass
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C:stableswap C:x/gamm Changes, features and bugs related to the gamm module.
Projects
Osmosis Chain Development
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[x/gamm][stableswap]: Update binary search upper bounds to reflect spec osmosis-labs/osmosis
1 participant
@AlpinYukseloglu