taking Price Impact into account for partial trade size#435
taking Price Impact into account for partial trade size#435rouzwelt wants to merge 13 commits into2026-01-30-raindex-router-modefrom
Price Impact into account for partial trade size#435Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Motivation
Caution
Do NOT merge before #430
This PR enables accounting for sushi's swap Price Impact for finding the partial trade size.
Solution
Checks
By submitting this for review, I'm confirming I've done the following: