-
Notifications
You must be signed in to change notification settings - Fork 201
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
Momentum Indicators - RSI, MACD ... (#120) #131
base: master
Are you sure you want to change the base?
Conversation
Implementation of RSI momentum indicator, ref -> #119
Hi @fmilthaler - Any updates on if/when you may merge this ? Thank you. |
Hey, sorry for the delay, I was away and then busy otherwise. |
Do you want me to modify the code ? |
Sorry for the delay @pythonhacker. I had to have a closer look at the RSI itself to understand it first.
For the macd, could you make use of the package mplfinance here instead? I don't like reinventing the wheel here. We can simply use one of the following:
to plot the macd. Again, sorry for the delay in this, I've been busy and pushed this back too many times. |
…for new type validations. Also changing docstrings.
b864b7c
to
1860dd8
Compare
@pythonhacker I've updated the code to be compliant with the recent master and made changes to the macd function to be like the example in mplfinance. Note: |
…t into feature/rsi-indicator
@fmilthaler - I have been out of touch for a while, due to work and just looked back at this. Didn't see the review request my bad. Kindly let me know if I can still review this. |
Implementation of RSI momentum indicator, thank you @pythonhacker. This closes #119.