Jimdandy wrote:I'm trying to figure out what i'm missing here...

if (EnforcePivotCross && (iOpen(NULL, PivotCrossTimeFrame, 1) <= DailyPivot || iClose(NULL, PivotCrossTimeFrame, 2) >= DailyPivot) ) SendLong = false;

We've got the current candle in progress.. 0
The last candle completed.. 1
and Candle before that 2

.....So I would think that if the last candle (1) opened below pivot and (1) closed above pivot that would be a cross for a long...

.....But if I understand this line.. (and apparently i don't) it says
If the last candle (1) opened below the pivot... OR if the candle before last (2) closed above the pivot... kill the long signal.

.....So if crossing candle is the last candle (1) it opens below the pivot before it crossed.... it would kill the long.

.....If one candle later when the crossing candle is now candle 2.... since it closed above the pivot it still would still kill the long.

.....Yet another candle passes, now the crossing candle is candle 3 .. out of the picture.. candle 1 opens above the pivot so it passes the test... however candle 2 closes above the pivot so again the long is killed......... there is something simple that I am missing here. How does it ever take a trade with this enabled?

It just seems to me that this says... you can take a long if candle 1 opened above the pivot and candle 2 closed below it. Which is kind of hard to do since candle 1 opens at the SAME PRICE that candle 2 closes. unless........ there is a gap over the weekend from the close of candle 2 to the open of candle 1
... or.... candle 2 is at the end of OLD day and candle 1 is above a NEW daily pivot.

Someone enlighten me please...... PipPip....Jimdandy

Jimdandy wrote:I can see why when you set it to true it greatly reduced the number of trades it took..... now can you explain that circle of fifths thingy to me? PipPip.. Jimdandy....

p.s. I have another ea that is doing the same thing with the stoploss error.. it's when it's moved to breakeven and is still trying to do it again even though the code plainly says RETURN if its already been done.... got to figure that one out next...

bool CloseEnough(double num1, double num2)
{
   /*
   This function addresses the problem of the way in which mql4 compares doubles. It often messes up the 8th
   decimal point.
   For example, if A = 1.5 and B = 1.5, then these numbers are clearly equal. Unseen by the coder, mql4 may
   actually be giving B the value of 1.50000001, and so the variable are not equal, even though they are.
   This nice little quirk explains some of the problems I have endured in the past when comparing doubles. This
   is common to a lot of program languages, so watch out for it if you program elsewhere.
   Gary (garyfritz) offered this solution, so our thanks to him.
   */
   
   if (num1 == 0 && num2 == 0) return(true); //0==0
   if (MathAbs(num1 - num2) / (num1 + num2) < 0.00000001) return(true);
   
   //Doubles are unequal
   return(false);

}//End bool CloseEnough(double num1, double num2)

Jimdandy wrote:........Yea I had an Aha moment when I read Gary's explanation of this problem and thanked him for it on down the thread.... but uh..... well..... I almost hate to bring it up.. but I think ........

.....if (EnforcePivotCross && iClose(NULL, PivotCrossTimeFrame, 1) >= DailyPivot) SendLong = false;

.....should be :

.....if (EnforcePivotCross && iClose(NULL, PivotCrossTimeFrame, 1) <= DailyPivot) SendLong = false;

.....if the last candle closed BELOW the pivot we should cancel the long should we not?

..... or have I just not had enough scotch yet to make this work? I mean if I have to drink scotch I'll do it... i'm willing to make that sacrifice..... the liquor store is only a half mile away and I have running shoes. :-D

Here I go....PipPip.....JimDandy.