Skip to content
This repository was archived by the owner on Feb 16, 2020. It is now read-only.

ReferenceError: config is not defined, with CCI from UI , fresh installation #1153

Closed
ghost opened this issue Sep 22, 2017 · 2 comments
Closed

Comments

@ghost
Copy link

ghost commented Sep 22, 2017

Note: for support questions, please join our Discord server

  • I'm submitting a ...
    [X] bug report
    [ ] feature request
    [ ] question about the decisions made in the repository

  • Action taken (what you did)

CCI Backtest on any exchange import (launched from UI) using default settings
CCI Paper Trader on exchange exchange (launched from UI) using default settings

Similar to #1117

Same fix worked

this.requiredHistory = config.tradingAdvisor.historySize; // line 28
Fix : this.requiredHistory = this.tradingAdvisor.historySize; // line 28

  • Expected result (what you hoped would happen)

Normal CCI Backtest result
Normal CCI Paper Trader behavior

  • Actual result (unexpected outcome)

Errors in both cases , only when using CCI

  • Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

Similar issue to #1117

`/home/user/gekko/strategies/CCI.js:11
this.requiredHistory = config.tradingAdvisor.historySize;
^

ReferenceError: config is not defined
at Base.method.init (/home/user/gekko/strategies/CCI.js:11:26)
at Base.bound [as init] (/home/user/gekko/node_modules/lodash/dist/lodash.js:729:21)
at new Base (/home/user/gekko/plugins/tradingAdvisor/baseTradingMethod.js:118:8)
at Actor.setupTradingMethod (/home/user/gekko/plugins/tradingAdvisor/tradingAdvisor.js:62:17)
at Actor.bound [as setupTradingMethod] (/home/user/gekko/node_modules/lodash/dist/lodash.js:729:21)
at new Actor (/home/user/gekko/plugins/tradingAdvisor/tradingAdvisor.js:23:8)
at load (/home/user/gekko/core/pluginUtil.js:95:22)
at /home/user/gekko/node_modules/async/dist/async.js:1156:9
at replenish (/home/user/gekko/node_modules/async/dist/async.js:1030:17)
at iterateeCallback (/home/user/gekko/node_modules/async/dist/async.js:1015:17)

Error: non-error thrown: Child process has died.
at Object.onerror (/home/user/gekko/node_modules/koa/lib/context.js:105:40)
at process._tickCallback (internal/process/next_tick.js:109:7)

/home/user/gekko/strategies/CCI.js:11
this.requiredHistory = config.tradingAdvisor.historySize;
^

ReferenceError: config is not defined
at Base.method.init (/home/user/gekko/strategies/CCI.js:11:26)
at Base.bound [as init] (/home/user/gekko/node_modules/lodash/dist/lodash.js:729:21)
at new Base (/home/user/gekko/plugins/tradingAdvisor/baseTradingMethod.js:118:8)
at Actor.setupTradingMethod (/home/user/gekko/plugins/tradingAdvisor/tradingAdvisor.js:62:17)
at Actor.bound [as setupTradingMethod] (/home/user/gekko/node_modules/lodash/dist/lodash.js:729:21)
at new Actor (/home/user/gekko/plugins/tradingAdvisor/tradingAdvisor.js:23:8)
at load (/home/user/gekko/core/pluginUtil.js:95:22)
at /home/user/gekko/node_modules/async/dist/async.js:1156:9
at replenish (/home/user/gekko/node_modules/async/dist/async.js:1030:17)
at iterateeCallback (/home/user/gekko/node_modules/async/dist/async.js:1015:17)`

@ghost
Copy link
Author

ghost commented Sep 22, 2017

Oops missed #1139

Sorry

@ghost ghost closed this as completed Sep 22, 2017
@askmike
Copy link
Owner

askmike commented Sep 25, 2017

Note that the fix is only applied to develop, it will be part of the next release!

This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant