-
Notifications
You must be signed in to change notification settings - Fork 184
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
Not getting expected CCI values #103
Comments
@MehdiAnis the fix is deployed to nuget, please update the package accordingly |
I think I interpreted the CCI Calculation Formula differently, thus my numbers are different. But I am not 100% sure which interpretation to take. I will dig in. But thanks for the Update, I will use it. |
@MehdiAnis Yes, I find that our interpretation are different. My version is getting the deviation using Abs(n-th TP - 14-th TP SMA) while yours is Abs(n-th TP - n-th TP SMA), I believe your version is a more accurate one, so I changed it. Most of the time, I find that there're some variations of formula for technical indicators, and I have to cross check the logic with different formula provide e.g. Wiki, Stockcharts, investopedia, etc. Anyway, I will end this issue. Thanks for your support on this library :) |
I used 200 made up data points to test CCI. My Data
High Low Close
1 1 1
2 1 1
3 1 1
4 1 1
200 1 1
in my excel calculation CCI values comes for each row = 66.66666667
Using candles.Cci(Period); gives me CCI = 123.80952380952380952380952381
I am attaching my excel file that I used for CCI calculation. Not sure why I am getting different result than the Trady CCI, any help appreciated. Thanks.
Trady_CCI_Check.xlsx
The text was updated successfully, but these errors were encountered: