-
Notifications
You must be signed in to change notification settings - Fork 1k
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
stdev does not take ddof = 0 #354
Comments
Hello @dnabb, Doh! Nice catch! In the mean time, edit your local copy of variance and stdev and replace that validation with the following until the code is updated. ddof = int(ddof) if isinstance(ddof, int) and ddof >= 0 and ddof < length else 1 Hope this helps! Kind Regards, |
Also I have added the changes to the development branch if you want to test it out. $ pip install -U git+https://github.com/twopirllc/pandas-ta.git@development Please let me know if it works as intended if you try the development branch. Thanks, |
Hello @dnabb, New stable version v0.3.14b should help resolve this Issue. Please let me know otherwise. $ pip install pandas_ta Thanks, |
Hello @dnabb, I assume by no response that the solution provided was sufficient. Thus I will be closing this issue in a few days. Kind Regards, |
Hello @dnabb, This should be fixed now. If you can test it out on the development branch and let me know, that would be great. $ pip install -U git+https://github.com/twopirllc/pandas-ta.git@development Thanks |
Hello @twopirllc , I can confirm the issue seems to be solved in 0.3.18b0. |
Just reopening this post. I observed TALIB But I just wanted to confirm with others that pandas_ta standard deviation should be safe to use right since it matches my final indicator calculations cross referencing tradingview. I will stop using talib for this function. code to try below
|
Closing this issue, I found the answer in the same talib post (did not read fully sorry!) While using TALIB need to add
|
Which version are you running? The lastest version is on Github. Pip is for major releases.
0.3.2b0
Describe the bug
Calling stdev with ddof=0 (and all other derived indicator, like bbands) actually returns a standard deviation with ddof = 1.
To Reproduce
Expected behavior
Should be the same as:
Additional context
The issue seems to be this validation which fails with ddof = 0.
The text was updated successfully, but these errors were encountered: