askmike / gekko

A bitcoin trading bot written in node - https://gekko.wizb.it/
MIT License
10.08k stars 3.93k forks source link

CLI Dev - ERROR: Please specify a config file. #627

Closed thegamecat closed 7 years ago

thegamecat commented 7 years ago

I have config.js but I'm guessing it's in the wrong place. Where should it be to support CLI backtesting?

askmike commented 7 years ago

I want to move towards the UI being the default way to use gekko. Gekko CLI is still supported, but you simple need to specify where the config is, for example:

node gekko --config config.js
thegamecat commented 7 years ago

So node gekko --backtest --config config.js ?

Understand the desire to use the UI but it would be great to keep the CLI as it actually provides more information and is easier to interact with when writing new strategies.

(it also currently supports backtests which the UI doesn't, as discussed)

askmike commented 7 years ago

You are right, and I will definitely keep the CLI version around.

I updated a lot of documentation in the develop branch: it suggest to run the UI on default, and CLI related stuff has been moved to to docs/advanced_usage. But I do need to write a document explaining how to run the CLI version (with config flag, etc).

(it also currently supports backtests which the UI doesn't, as discussed)

I want to merge dev into stable soon, and this is one of the few things I still need to fix before I can do that. Note that the talib bug is fixed now (you can now run talib-macd in the UI - if you have talib installed, just not other custom methods).

thegamecat commented 7 years ago

Yep, don't worry about that issue, very low priority, and the date range feature will fix it in a much more practical way.

The talib bug has been fixed for some time which is absolutely brilliant.

askmike commented 7 years ago

The talib bug has been fixed for some time which is absolutely brilliant.

So last time you mentioned it you needed to enable talib in a hidden config (baseConfig.js), but this is not necessary anymore now :)

thegamecat commented 7 years ago

Ah, you're a ninja!

askmike commented 7 years ago

and the date range feature will fix it in a much more practical way.

This is included in the latest develop now, the list is getting shorter :)

thegamecat commented 7 years ago

Amazing thanks!