You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
There are 2 ways to normalize ATR, namely normalizing before averaging and normalizing after averaging. ref https://www.macroption.com/normalized-atr/.
By reviewing the code, both natr and atr(percent=True) results in normalizing after averaging.
Expected behavior
It may be better to have atr(percent=True) to normalize after averaging and natr to normalize before averaging. Or else there is a duplication of implementation, one of them should be removed to remove confusion.
The text was updated successfully, but these errors were encountered:
Which version are you running? The lastest version is on Github. Pip is for major releases.
Do you have TA Lib also installed in your environment?
Have you tried the development version? Did it resolve the issue?
Same issue in dev.
Describe the bug
There are 2 ways to normalize ATR, namely normalizing before averaging and normalizing after averaging. ref https://www.macroption.com/normalized-atr/.
By reviewing the code, both
natr
andatr(percent=True)
results in normalizing after averaging.To Reproduce
cmp_pandas_ta_natr_atrp.ipynb.zip
Expected behavior
It may be better to have
atr(percent=True)
to normalize after averaging andnatr
to normalize before averaging. Or else there is a duplication of implementation, one of them should be removed to remove confusion.The text was updated successfully, but these errors were encountered: