Fratelli EA by Dragos and magft

[MickyMouse]

hi guys... and girls


if (MACD1 > (0+2*Point) || MACD0 >= (0+2*Point)) MACD = up;
if (MACD1 < (0-2*Point) || MACD0 <= (0-2*Point)) MACD = down;

please modify the ea . dont use any more buffer for macd . up is up and down is down
as in above 0 is up, bellow 0 is down D:

[magft]

hi guys...


if (MACD1 > (0+2*Point) || MACD0 >= (0+2*Point)) MACD = up;
if (MACD1 < (0-2*Point) || MACD0 <= (0-2*Point)) MACD = down;

please modify the ea . dont use any more buffer for macd . up is up and down is down
as in above 0 is up, bellow 0 is down D:

Thanks for the update, currently fixing a few issues.

There is an obvious copy and paste error in LookingForTradingOportunities()

Code: Select all

//Short
   //if ((((Open[1] > MAVal && Close[1] <= MAVal) || (Close[2] > MAVal && Open[1] <= MAVal && Close[1] < MAVal)) && trend == down) 
   //|| (Close[1] <= MAVal && direction == down) && ArrowBar>LastOrderTime)// && Open[1] > MAVal)
   if(Close[1] < MAVal && MACD == down && Signal == down && direction == down && (ArrowBar>LastOrderTime||SARSignal==down))
   //if(Close[1] < MAVal && MACD == down && Signal == down && direction == down && SARSignal==down)
   {
      if (!TradeShort) return;
      //if (Close[1]>Open[1]) return;
            
      type = OP_SELLSTOP;
      //Check if use HighLow or just Close
      if (UseHigh_Low) price = NormalizeDouble(High[2] - (SellBufferPips*Point),Digits);
      else price = NormalizeDouble(Close[2] - (SellBufferPips*Point),Digits);

Should have the High[2] replace with Low[2]

Code: Select all

//Short
   //if ((((Open[1] > MAVal && Close[1] <= MAVal) || (Close[2] > MAVal && Open[1] <= MAVal && Close[1] < MAVal)) && trend == down) 
   //|| (Close[1] <= MAVal && direction == down) && ArrowBar>LastOrderTime)// && Open[1] > MAVal)
   if(Close[1] < MAVal && MACD == down && Signal == down && direction == down && (ArrowBar>LastOrderTime||SARSignal==down))
   //if(Close[1] < MAVal && MACD == down && Signal == down && direction == down && SARSignal==down)
   {
      if (!TradeShort) return;
      //if (Close[1]>Open[1]) return;
            
      type = OP_SELLSTOP;
      //Check if use HighLow or just Close
      if (UseHigh_Low) price = NormalizeDouble(Low[2] - (SellBufferPips*Point),Digits);
      else price = NormalizeDouble(Close[2] - (SellBufferPips*Point),Digits);

I have done a few other fixes for the next version but that is quite an important one!! The EA is steadily making profit everyday so cant complain

Mike

[MickyMouse]

you guys are doing a great job keeping it updated and not discontinued..
congrats on that ...

[DING]

my testing upgrade to Fratelli V2.6 yesterday, But so quiet today , not open any order till now,
what wrong for me? :

[magft]

my testing upgrade to Fratelli V2.6 yesterday, But so quiet today , not open any order till now,
what wrong for me? :

Not sure, i have had loads of trades today including catching the ride up on the EU. Do you have the smiley face in the corner?

[magft]

Update in post #1

[Xfinity]

Cheers, will test it asap. Is there any other file besides the ones in the first post I need? I am going to set-up a new computer to test more bots. Thanks!

[magft]

Cheers, will test it asap. Is there any other file besides the ones in the first post I need? I am going to set-up a new computer to test more bots. Thanks!

Nope, unless you want to visually test the EA as well then you need all the indicators from the Fratelli thread.

[Xfinity]

Thanks for the quick reply, it looks a little nicer now - I was in the process to add all indis I had on my main computer.

[blahn]

Here's the last 5 or 6 hours of 2.7s POs compared to the main indi. about 50 percent of the Pos match the main indi.

au27.gif

eu27.gif

ej27.gif