We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
A liquidityNet map in a concentrated pool should be a int->int (or maybe int->big_int). It can be negative too:
services/crates/solvers/openapi.yml
Line 348 in a72c4c3
Can lead to reverts because solvers have the wrong AMM state.
The text was updated successfully, but these errors were encountered:
I think this is mainly a documentation bug. Looking at the dto, the type is actually signed
services/crates/solvers/src/api/routes/solve/dto/auction.rs
Lines 347 to 348 in bffb3cb
So it shouldn't lead to reverts.
Sorry, something went wrong.
Oh great, I thought these API docs were autogenerated. Closing.
[Trivial] fix solver docs (#2240)
1cadc7b
Fixes #2229 (cf. discussion there on why this is correct)
64f9363
Successfully merging a pull request may close this issue.
Problem
A liquidityNet map in a concentrated pool should be a int->int (or maybe int->big_int). It can be negative too:
services/crates/solvers/openapi.yml
Line 348 in a72c4c3
Impact
Can lead to reverts because solvers have the wrong AMM state.
The text was updated successfully, but these errors were encountered: