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

Limit orders on dydx #33

Open
ypsik opened this issue Jun 18, 2024 · 3 comments
Open

Limit orders on dydx #33

ypsik opened this issue Jun 18, 2024 · 3 comments

Comments

@ypsik
Copy link

ypsik commented Jun 18, 2024

Hello,

i tested your connector, great job, just i have one issue.
You implemented dydx connector as market orders,

const type = OrderType.MARKET;

its executing to random prices and not on prices you set as limits.
Because markets on dydx are often not liquid Its just losing a lot of money, Maybe you can switch to limit orders? Maybe as option in config ?
I think you need just change from market to limit this line.

@oh92
Copy link

oh92 commented Oct 3, 2024

Hey @ypsik, @junta
Thank you for bringing this up, as DYDX often encounters issues executing market orders even on the UI side.
By enabling LIMIT, orders this would not only reduce slippage like @ypsik mentioned, but also increase the order success rate.

On my end, there seems to be fewer issues entering a position, however, position exits often fail.
For example, the websocket signal is sent via TradingView, received and recorded in the tradeHistoryPerpetual.csv, however, no action at all is taken on exchange.

For my setup, I am currently running on Render, but am not sure how to fetch a more detailed log history.

Recent signal sent that resulted in no action:
image

@oh92
Copy link

oh92 commented Oct 4, 2024

Hey @ypsik,
I do not have permission to commit to this repo, but I have forked and modified the dydx service file to support limit orders under my own branch.
I will work on adding additional configuration options within the alert settings for more support.
https://github.com/oh92/tradingview-alert-connector/tree/dydx-enable-limit-orders

@junta
Copy link
Owner

junta commented Oct 4, 2024

Hey @oh92,
Can you open Pull Request if you don't mind?
I'll merge your limit order work(and other improvement too) to this repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants