Pyramid Trading - Scaling into a position - DCA Trading possible > Possible way #1113
Conversation
No work should be done in this PR anymore, because the basis and the basic assumptions (about how to solve this) is simply not good. Everything would have to be tackled completely differently in order to solve this problem, so this PR cannot be improved in this way either. I do not agree with this PR. It is nothing else than a workaround which pollutes a lot of our base classes only because of one concept. It is by far not a clean solution and would worsen the condition of the lib, since one concept was sloppily crammed into other concepts. @wesssup If it works for you, then leave it in your branch. I will not agree to put this into this lib. It would be far better to provide a clean solution to this problem as suggested here: #1054 (comment). If #1054 was done, then this "pyramid trading" would end up being nothing more than a criterion type indicator (called This PR is nothing more than a workaround and is far from a clean and future-proof solution - it's a typical hack and pollutes the lib. |
|
--> Pls read ticket #1106 I already provided an answer ... I don't agree on the hack part , as you will need to change the core logic, I don't see how you can move otherwise from allowing only one position to multiple ... needed for scaling into a bigger transaction. But true, as I mentioned above ... there's still work to be done ... BUT we did some tests in comparison with Trading view Pine back testing ... and we got the same results with this code ... |
Possible approach :#1106
Allowing Pyramid Trading - Scaling into a position - DCA Trading possible > Possible way
Still work to be done !!!!
--> Not for MERGE on head branch ! -> Dev still needed see ticket for more info