joeldg / bowhead

PHP trading bot framework
Apache License 2.0
799 stars 289 forks source link

bowhead_ema_candle ema is always 0 #32

Open lploumen opened 7 years ago

lploumen commented 7 years ago

Hi, Shouldn't this line
https://github.com/joeldg/bowhead/blob/ee67a6ef0a6f82100e1ac7940a4dad172a9a8bd1/app/Traits/CandleStrategies.php#L24 be changed to requires $ema = trader_ema($data['close'], 20); The actual version always compute an exponential moving average of 0.

vitoo commented 6 years ago

I think yes it should be changed