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

[speed] Improve execution time for small orders #4736

Open
anxolin opened this issue Jul 26, 2024 · 1 comment · May be fixed by #4911
Open

[speed] Improve execution time for small orders #4736

anxolin opened this issue Jul 26, 2024 · 1 comment · May be fixed by #4911
Labels
Planned Just a label to easily see some of the planned things for the quarter easily

Comments

@anxolin
Copy link
Contributor

anxolin commented Jul 26, 2024

Potential ideas:

  • Detect when gas price changes might have a big impact on the execution
  • React to the above (preventing the user from creating the order in some cases and allowing to place anyway knowing it might be delayed), alternatively or in addition, make sure if the user places the order we recommend adding some extra buffer for the slippage tolerance
  • Potentially, the smart slippage calculation should take volume into account
  • Good to have: estimate the expected volatility in gas prices for the next 30min and enhance the smart slippage calculations

After the improvements, we should monitor the SLI dashboard and focus on understanding the outliers.

@anxolin anxolin added the Planned Just a label to easily see some of the planned things for the quarter easily label Jul 26, 2024
@anxolin anxolin changed the title [speed] Smart Default Slippage [speed] Improve execution time for small orders Jul 26, 2024
@elena-zh
Copy link

Hey @anxolin , should #4316 task be closed then?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Planned Just a label to easily see some of the planned things for the quarter easily
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants