Description
Note: this is the technical bug tracker, please use other platforms for getting support and starting a (non technical) discussion. See the getting help page for details.
I'm submitting a ...
[X ] bug report
[ ] question about the decisions made in the repository
newest Stabe build
Action taken (what you did)
Attempted to add multiple tulip indicators (any tulip indicators).
Then run strategy via command line.
Expected result (what you hoped would happen)
The method.update function will be called.
Actual result (unexpected outcome)
The method.update function is not called.
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
You can mix and match any of the tulip indicators. Declaring multiple of them just does not work.
this.addTulipIndicator('_mfi', 'mfi', {optInTimePeriod: 14});
this.addTulipIndicator('_bb', 'bbands', {optInTimePeriod: 20, optInNbStdDevs:2});
this.addTulipIndicator('_stoch', 'stoch', {optInFastKPeriod: 3, optInSlowKPeriod: 3, optInSlowDPeriod: 3});
this.addTulipIndicator('_macd', 'macd', { optInFastPeriod: 10, optInSlowPeriod: 21, optInSignalPeriod: 9});
Activity
askmike commentedon Sep 19, 2018
Thanks for the report, will check asap.
bvanhou commentedon Sep 19, 2018
Thanks, this unfortunately is a big show stopper. I just tested and found that the issue occurs in v0.6.6 and not v0.6.5. Just verified.
#if this helps
config.tradingAdvisor = {
enabled: true,
method: 'tulip-multi-strat',
candleSize: 1,
historySize: 10,
}
**Note: Issue will always occur doesnt matter what you choose
eusorov commentedon Sep 30, 2018
I can confirm, in standalone mode nothing happens but with UI-Mode it works like expected.
Added the 4 tulip indicators to tulip-multi-strat.js from above and the update method is called as expected.
In general it is probably difficult to see logs in console, because log.js prevents in UI-mode. But if you add to _write method a console.log(message); like this:
than you see that the update method is called.
bugfix askmike#2548 if no strategy.historySize found, get from tradin…
eusorov commentedon Oct 3, 2018
In UI mode there is :
candleSize = 1
historySize = 4
But in standalone mode, in sample-config.js there is no such settings for tulip-multi-strat.
With the bugfix we check if historySize was provided or not.
bugfix #2548 if no strategy.historySize found, than get it from tradi…
stale commentedon Dec 2, 2018
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. If you feel this is very a important issue please reach out the maintainer of this project directly via e-mail: gekko at mvr dot me.