SteveHopwood » Sun May 27, 2018 10:55 pm wrote:I just realised that I have failed to acknowledge a contribution by Brenden last week. He sent me this pm:The latest version includes this fix. DIYers need to add it.odrisb wrote:Hi Steve
Noticed this issue when I set DeletePendingTradesOnOppositeColour = true;
The EA was deleting the pending orders but the ReadIndicatorValues() function was adding them back in again in an endless loop.
I made the following changes in the ReadIndicatorValues() which I believe may help:
Line 2170
if ((BuyStopsCount > 0 || MarketBuysCount > 0) && (!DeletePendingTradesOnOppositeColour || HtfSsStatus[PairIndex] != red))
Line 2210
if ((SellStopsCount > 0 || MarketSellsCount > 0) && (!DeletePendingTradesOnOppositeColour || HtfSsStatus[PairIndex] != blue))
I have not tested extensively but it seemed to stop the looping![]()
Cheers
Brenden
Thanks Brenden.![]()
![]()
![]()
![]()
![]()
![]()
And here is the importance of always updating to the latest available version...
I had 5 demos still running on v1i and I got my but kikked from GP that in the end was forced to ban my IP from their demo servers.
Combined these demos were doing over 1 million order modifications a day.
At the beginning I thought setting MinMinutesBetweenTrades to 1 minute was going to be enough, but I was wrong; v1i got me banned twice, v1l would have not.
An honorable mention to GP and the way they have managed my requests.
They were simply AMAZING! :hi:
Despite being on demo they always gave me full attention and were always quick in answering my doubts and restoring my demo functionality.
Thank you Jeremy, thank you GP!!!