-
Notifications
You must be signed in to change notification settings - Fork 4k
Support tulip indicators (windows TA library) #708
Comments
Impressive looking at the benchmarks and well documented too... +1! |
So I agree with doing this but....from my testing on windows, just run gekko in bash on windows 10 then talib is fine. If you use SQLite then go to the file gekko/plugins/sqlite and change the line 53 from: to db.run("PRAGMA journal_mode = DEL"); And if you're copying SQLite dbs from another install of gekko then make sure you change the db pragma journal mode to DEL in a tool like DB Browser for SQLite. I would love to know the performance difference between DEL and WAL or even if they are used, but I have a 500meg btcusd dataset and it seems to make no difference. All that said, the more indicators the merrier, I am personally unsure if it should be a priority, but I'd be happy with it integrated. Edit; just reviewed the performance....this would be marvelous to implement! +1 from me. |
+1 |
+1 |
+1 Maybe we should add instructions to the docs to get gekko running on bash on Windows 10 |
+1 |
I need awesome oscillator but Ta-Lib doesn't have one. |
All thanks to @cmroche this is now available in the develop version of Gekko! (download here: https://github.com/askmike/gekko/tree/develop - using that link you can also browse the It will be available in the next release of gekko! As such I am closing this issue already :) |
+1 |
@dyeghikoo tulip has been available for a while now, see here: https://gekko.wizb.it/docs/strategies/tulip_indicators.html |
If it is implemented now, can please the installation guides be updated? |
Hi, |
+1 |
Gekko supports TA-lib, a well known technical analysis library that supports a lot of different indicators. However it is very hard to compile this library on Windows (which Gekko supports).
Tulip indicators is a relative new library that supports roughly the same indicators. The node wrapper just added precompiled windows binaries which makes installing on windows a breeze.
Please +1 this if you want this implemented, will work on this regardless but might prioritise if a huge number of people want this.
The text was updated successfully, but these errors were encountered: