thegame,
ok, so what i see so far it's darn close. If you have time, maybe you can explain to me why this line gets commented out:
/*if (TimeMinute(TimeCurrent()) != 0)*/ ClearSymbolMarks();
I took that to mean that every hour, it clears the previous trades that were added in the global variables, so it can look again to see who the biggest winner/loser is and whether there is a previous trade that is profitable in order to add. So it didn't dawn on me to just comment it out, thought it was where I had to add the 30min check and have it clearmarks at 0 and 30.
Not sure if that is the issue that I see so far, but I believe it was what someone fixed for me last time? What I saw instantly was eurjpy being the biggestwinner on it's first check, it opened the ej trade twice. Other than that, looks like its working well, but it'd be nice to not allow it to open a trade more than once per check.
The prior fix was when I had it set to 0 || 30 instead of 0 && 30, so maybe the same issue happening here?
ok, so what i see so far it's darn close. If you have time, maybe you can explain to me why this line gets commented out:
/*if (TimeMinute(TimeCurrent()) != 0)*/ ClearSymbolMarks();
I took that to mean that every hour, it clears the previous trades that were added in the global variables, so it can look again to see who the biggest winner/loser is and whether there is a previous trade that is profitable in order to add. So it didn't dawn on me to just comment it out, thought it was where I had to add the 30min check and have it clearmarks at 0 and 30.
Not sure if that is the issue that I see so far, but I believe it was what someone fixed for me last time? What I saw instantly was eurjpy being the biggestwinner on it's first check, it opened the ej trade twice. Other than that, looks like its working well, but it'd be nice to not allow it to open a trade more than once per check.
The prior fix was when I had it set to 0 || 30 instead of 0 && 30, so maybe the same issue happening here?