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

[Bug]: EstimateAMMBounds API doesn't not match up with the state of actual AMM's on stagnet. #11526

Closed
wwestgarth opened this issue Aug 6, 2024 · 0 comments · Fixed by #11534

Comments

@wwestgarth
Copy link
Contributor

Problem encountered

Taking an AMM currently active on stagnet and passing its definition through the EstimateAMMBounds API shows that the AMM's position is outside of the max/min position reported by the API.

My hunch is that the API is wrong, and it is not taking into account the market's price/position decimals.

Observed behaviour

The EstimateAMMBounds API does not reflect real life.

Expected behaviour

The EstimateAMMBounds API returns correct results.

Steps to reproduce

1. Submit `xyz` transaction
2. With this payload
3. Query `abc` API
4. Observe the error

Software version

develop

Failing test

No response

Jenkins run

No response

Configuration used

No response

Relevant log output

No response

@wwestgarth wwestgarth added this to the 🏯 Suzuka Castle milestone Aug 6, 2024
@wwestgarth wwestgarth self-assigned this Aug 6, 2024
@wwestgarth wwestgarth moved this to In Progress in Core Kanban Aug 6, 2024
@vega-issues vega-issues moved this from In Progress to Waiting Review in Core Kanban Aug 7, 2024
@vega-issues vega-issues moved this from Waiting Review to Approved in Core Kanban Aug 7, 2024
@vega-issues vega-issues moved this from Approved to In Review in Core Kanban Aug 7, 2024
@vega-issues vega-issues moved this from In Review to Approved in Core Kanban Aug 7, 2024
@vega-issues vega-issues moved this from Approved to In Review in Core Kanban Aug 7, 2024
@vega-issues vega-issues moved this from In Review to Approved in Core Kanban Aug 7, 2024
@github-project-automation github-project-automation bot moved this from Approved to Merged in Core Kanban Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant