search

kylejusticemagnuson / pyti

Python library of various financial technical indicators
MIT License
656 stars 164 forks source link

Incorrect Stochastic Oscillator? #16

Closed slidenerd closed 6 years ago

slidenerd commented 6 years ago

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 commented 6 years ago

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

RichardDale commented 4 years ago

Fix inside #29