-
Notifications
You must be signed in to change notification settings - Fork 2k
Enhance backtester: support peroid, min_peroids param and rsi_macd strategy #368
Conversation
Do you have some articles about rsi_macd / Stochastic RSI strategy ? |
Just some simple tutorials:
|
Hi, it was really my first PR and i screwed up the naming. Indeed the srsi_macd is a better name. BTW i'm not familiar with this backtester, what does it do and do i use it? |
Hi @talvasconcelos, thanks for your code. So you could decide the name together with @carlos8f :) And
After all tests finished, it will generate a result But the default option ranges could not cover all cases, so we need custom them in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 useful stuff
This is awesome... i'll make a PR today with changed name then. I'm also making a new strategy with CCI and SRSI than i hope to push today. This tool is awesome as i run endless sims testing different parameters!! Hope it gets merged soon so i can test it. Thanks for this |
sent the pr with the strategy name change, guess it screws your work, sorry man. also fixed sma, take a look and see if it good like that. hope today I'll push a new strat... |
Just fine, I will review the code soon. |
…rategy (DeviaVir#368) * backtester: Support period and mini_periods * backtester: Support rsi_macd strategy
The new added
rsi_macd
strategy setmin_peroids
to 200 as default value which is different than others, so I support it in backtester firstly.BTW, looks
srsi_macd
is a better name thanrsi_macd
for it use Stochastic RSI in fact...