search

freqtrade / freqtrade

Free, open source crypto trading bot
https://www.freqtrade.io
GNU General Public License v3.0
28.88k stars 6.18k forks source link

Kucoin futures integration #10750

Closed TheyCallMeSticky closed 1 month ago

TheyCallMeSticky commented 1 month ago

Describe your environment

(if applicable)

Describe the enhancement

Explain the enhancement you would like

Hello !

I want to integrate Kucoin Futures support into freqtrade. On CCXT, it's a separated exchange and it's supported.

As a developer, i can participate.

xmatthias commented 1 month ago

Kucoin is an exchange that's not officially supported because it's API is known to be flaky / unreliable. Trying to Supporting that actively is a nightmare.

From what i understood, their Futures API is worse from a stability / reliability point of view - so i think this will be time wasted (especially in the maintenance/support phase).

While in theory - it could be added in a "community tested" approach (similar to spot) - that's a bit more difficult for futures, as that requires quite additional code and logic, which will need both maintenance (also going forward), and also eventual help in case people have problems.

While i appreciate the incentive to try and contribute - I don't think it's a good idea at this point.

It's not personal against you - however you'd be a new contributor - who never interacted with this repo before. So it's becomes a question of trust to some degree. It's unknown if you'll stick around (for how long? how much time a week / month can you dedicate to eventually support users), how responsive you'd be in case of questions, ...

Assuming you'd lost interest in X months time there'd be 2 options it'd either fall back to existing contributors / maintainers to try and support an exchange we know is flaky before even trying to implement it - which quite honestly, I've got no will to do for an exchange i know doesn't work well. The alternative is to simply remove that support again (remove the code contributed, ...) - but then it'd be even more time wasted, both on your side trying to add it - and on our side reviewing code, merging, ...

TheyCallMeSticky commented 1 month ago

Ok, i understand why it's not a good idea to open it as an official release. I'll do it on my side. Too much problems

xmatthias commented 1 month ago

what you can always do is let us

We can then still determine if it makes sense to include it or not ;)

TheyCallMeSticky commented 1 month ago

As European laws tends to complicate futures trading for their inhabitants. They should ban kucoin also one day.

I'll move to hyperliquid. It's supported by CCXT. No kyc, DEX.

Did you already try to implement it ? Any information about stability / security ?

xmatthias commented 1 month ago

They should ban kucoin also one day.

That's not how it works - technically, futures trading is banned on a country by country basis. It's the exchanges that decide to either follow or ignore these regulations (obviously countries going after specific exchanges can change their mind - but it's not a "the country said, exchange X is banned from using futures" topic). :laughing:

for hyperliquid - best follow #10377. There's some work done on that - and it seems to work mostly fine with very recent versions of ccxt - but so far, testing for spot is ongoing, with issues being found "weekly" (more or less) :laughing: