Skip to content
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

Optional config option for EMA numbers #266

Open
oestevan8 opened this issue May 27, 2021 · 4 comments
Open

Optional config option for EMA numbers #266

oestevan8 opened this issue May 27, 2021 · 4 comments
Labels
feature request Feature request for roadmap

Comments

@oestevan8
Copy link

Describe the solution you'd like
For example, for the "config" options:

"emabuy" : 12, 26,
"emasell": 8, 12,
..
Thank you

@whittlem
Copy link
Owner

Why the shorter moving average on the sell? Is it to detect the change faster?

@whittlem whittlem added the feature request Feature request for roadmap label May 27, 2021
@whittlem
Copy link
Owner

I've added "ema8" to the data frame and the crossovers with "ema12" which already existed. They are called "ema8gtema12", "ema8gtema12co", "ema8ltema12", and "ema8ltema12co". I've not done any integration into the bot yet because it's quite a lot of work and I need to see how many others want to to determine priority.

If you want to try the feature now just do a find and replace in pycryptobot.py for "ema12ltema26co" with "ema8ltema12co". That is the quick way to try your feature out now.

@oestevan8
Copy link
Author

I did hundreds of test on TradingView with different EMA indicators, and by changing EMA sell and buy signal numbers, many times I get slightly better result.
For each bot I start I would use different numbers according to test result. (Those 8, 12 are just example, probably not the best for a sell signal).
I'll try your suggestion, thank you!

@yanone
Copy link

yanone commented May 28, 2021

If this comes to pass as a setting, I would like to appeal to make the new setting a default for Binance, and the old settings optional, if they are indeed found by you and the other trading experts to be beneficial.

Personally, I’m not a trading expert, so I just want to generally appeal to make as much trading intelligence as possible (and improvements thereof) a default functionality rather than optional.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Feature request for roadmap
Projects
None yet
Development

No branches or pull requests

3 participants