-
Notifications
You must be signed in to change notification settings - Fork 566
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
Bollinger Bands - use EMA? #213
Comments
BB uses SMA, currently it is not possible to use EMA, pull request welcome, similar feature is implemented in MACD, https://github.com/anandanand84/technicalindicators/blob/master/src/moving_averages/MACD.ts#L31 |
I tried installing the package but it broke at canvas. Is there a lightweight way of adding tests? If I just add an option to an indicator, seems like a test should just compare the output to an array. |
@tredondo what OS are you using to do development? I had the same issue as you did getting canvas installed. Currently using node v12.16.3 on macOS Catalina. I've played around with the packages.json file a little and I think I got it working but can't seem to get things rolling. Let me know. Thanks, |
You can just copy the original script and change SMA to EMA on the 6th line. //@Version=4 |
What type of MA does BB use? I don't see any docs.
I assume it uses SMA? Is there an option to use EMA?
The text was updated successfully, but these errors were encountered: