-
Notifications
You must be signed in to change notification settings - Fork 468
fix: dust protection calculation optimization #2830
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| [SupportedChainId.SEPOLIA]: '', | ||
| }; | ||
|
|
||
| export const DUST_PROTECTION_MULTIPLIER = 1.001; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👉 Conservative market
Max APR ≈ 80%
Max per hour: 0.91 bps
👉 Aggressive / volatile market
Max APR ≈ 300%
Max per hour:≈3.4 bps
👉 Extreme custom strategy (rare but allowed)
Max APR ≈ 1000%
Max per hour:≈11.4 bps
We have 10 min max now.
c14c3b0 to
26532f2
Compare
|
📦 Next.js Bundle Analysis for aave-uiThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
|
📦 Next.js Bundle Analysis for aave-uiThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
The dust protection (which was overestimated) was affecting the quote received only in one side, so for big orders where our overestimated costs are not significant and where slippage were low, the market orders were struggling more to get filled.