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

config.normal exchange setting necessary #112

Closed
plangin opened this issue Jan 3, 2014 · 4 comments
Closed

config.normal exchange setting necessary #112

plangin opened this issue Jan 3, 2014 · 4 comments
Labels

Comments

@plangin
Copy link

plangin commented Jan 3, 2014

/ docs / Configuring_gekko.md states:

"The advanced zone

If you want to monitor an exchange but act on a different one, or watch a single exchange and act on multiple exchanges based on the advice you have to disable (or uncomment) the normal zone and configure the advanced zone:"

Almost!

My setup is: Watching Mt.Gox, trading at Mt.Gox and BTC-E.

I need to leave at least in this case:

config.normal = {
}

Otherwise Gekko throws this error:

"TypeError: Cannot read property 'exchange' of undefined
at null. (/Users/plangin/Desktop/gekko-mtgox-btce-live/methods/exponential-moving-averages.js:91:21)
at /Users/plangin/Desktop/gekko-mtgox-btce-live/node_modules/lodash/dist/lodash.js:901:23
at Function.forEach (/Users/plangin/Desktop/gekko-mtgox-btce-live/node_modules/lodash/dist/lodash.js:3297:15)
at TradingMethod.calculateEMAs (/Users/plangin/Desktop/gekko-mtgox-btce-live/methods/exponential-moving-averages.js:90:5)
at bound as calculateEMAs
at EventEmitter.emit (events.js:92:17)
at bound as emit
at CandleCalculator.calculateCandle (/Users/plangin/Desktop/gekko-mtgox-btce-live/realtimeCandleFetcher.js:177:8)
at bound as calculateCandle
at null. (/Users/plangin/Desktop/gekko-mtgox-btce-live/realtimeCandleFetcher.js:70:12)"

The manual must be more specific here.

@askmike
Copy link
Owner

askmike commented Jan 3, 2014

Good catch, I am happy to accept pull requests of updated docs.

@plangin
Copy link
Author

plangin commented Jan 6, 2014

I have no idea how to do this. Just see me as a normal user / tester. ;-)

@askmike
Copy link
Owner

askmike commented Jan 6, 2014

This is definitely an outdated doc, though right now I want to focus on a new version that is hopefully going to be out soon. That one will feature new rewritten docs.

@askmike
Copy link
Owner

askmike commented Feb 11, 2014

This has been fixed since the new version.

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

No branches or pull requests

2 participants