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

Incorrect Stochastic Oscillator? #16

Closed
slidenerd opened this issue Jun 9, 2018 · 2 comments
Closed

Incorrect Stochastic Oscillator? #16

slidenerd opened this issue Jun 9, 2018 · 2 comments

Comments

@slidenerd
Copy link

First of all, thank you for the awesome library

I noticed that stochastic oscillator needs h,l,c
http://incrediblecharts.com/indicators/stochastic.php
http://stockcharts.com/school/doku.php?id=chart_school:technical_indicators:stochastic_oscillator_fast_slow_and_full
For some reason it seems that you are not using high and low but rather the highest and lowest values from the close. Am I missing something? Would appreciate some clarification on the same

@kylejusticemagnuson
Copy link
Owner

Hi, @slidenerd
You are correct. It should be using the high and low, thanks for submitting the issue.

@RichardDale
Copy link

Fix inside #29

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

3 participants