-
Notifications
You must be signed in to change notification settings - Fork 57
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
Feature: ability to add rules to a list #14
Comments
Hello! I've had a thought on turning this EA into a sort of a programmable environment where conditions, actions, and filters on the platform, account, trades, and symbols could be arbitrarily combined to create systems of any complexity. And I thought that quite doable. However, I'm not sure that it's a good idea to turn the AP into that - it's best to start a new EA for that. And also, that's a lot of coding. So, I don't know when (and if) I'll start on this. PS: Your condition 6 is already there as 'Equity <= % of balance' is the same as 'Floating loss rises to % of balance'. You just adjust the % by 100 - %. For example, if you want 'Equity <= 10% of balance', you set 'Floating loss rises to 90 % of balance'. PPS: 'Margin Level <= %' is there. |
Hi, OK, do let me know on this issue if you do complete this, even if it's in a new EA (so not in this project)! Ah, thank you re condition 6! Good to know! I don't see 'Margin Level <=%'. |
You have to enable 'Margin Level <=%' via input parameters. If you set the 'Disable margin level less or equal condition.' input parameter to false, the condition will appear on the Conditions tab. |
Oh, great, thank you! I hadn't spotted that. |
Hi, right now, I can see I might need to run 8 or so instances of this EA on my terminal, because I'd want rules for:
So, I was wondering if it would make sense instead to have an Add rule button, and the rule appears below the checkboxes in a little scrollable panel, or in a pop-up window, and one could Edit or Delete rules from there, and basically have multiple rules running per instance of the EA.
I appreciate this may add quite a lot of complexity to the EA, and therefore may not be something you want to do, but I put it out there in case!
Cheers!
The text was updated successfully, but these errors were encountered: