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

tulindError: needs to be number #47

Open
aketekt opened this issue Jun 14, 2018 · 1 comment
Open

tulindError: needs to be number #47

aketekt opened this issue Jun 14, 2018 · 1 comment

Comments

@aketekt
Copy link

aketekt commented Jun 14, 2018

I'm using strat NEO..

Runs well in Gekko UI but get the follow error in gekkoga.. Not sure if it's user error or bug..

The Talib error get thrown also when run successfully in gekko ui.. " TALIB indicators could not be loaded, they will be unavailable." Which makes me think this is not my issue.

So Seems like it's specifically not passing a value to gekko params throw tulindError + paramName + ' needs to be a number';

Would really appreciate some help.

Config is

conf.txt

Error :
err.txt

@aketekt
Copy link
Author

aketekt commented Jun 14, 2018

hmmmmm.... I added a try catch to the strat and it fixed my issue... That's kinda wacky..
try {
this.addTulipIndicator('maSlow', 'sma', { optInTimePeriod: this.settings.SMA_long });
this.addTulipIndicator('maFast', 'sma', { optInTimePeriod: this.settings.SMA_short });
this.addTulipIndicator('BULL_RSI', 'rsi', { optInTimePeriod: this.settings.BULL_RSI });
this.addTulipIndicator('IDLE_RSI', 'rsi', { optInTimePeriod: this.settings.IDLE_RSI });
this.addTulipIndicator('BEAR_RSI', 'rsi', { optInTimePeriod: this.settings.BEAR_RSI });
this.addTulipIndicator('ROC_val', 'roc', { optInTimePeriod: this.settings.ROC });
} catch (error) {
console.log(error);
}

My guess is that gekkoga is trying to start papertrader.js before advisor has finished so it doesn't have the optInTimePeriod and throws error..As we are async.. Maybe try catch slow it down enough to get the results first... Or try catch prevent an blocking error somehow. ...(edited)

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

1 participant