SAFT EA MOD, HELP NEEDED PLEASE

[Anonymous]

Hi Steve

I am trying to modify your SAFT helper ea to test LDHT's settings (break of 50 SMA by price and break of 21 LWMA by OBV).

I have modified the buy/sell code as follows

//Long trade
//Trendline colour
if (!TradeLong) SendLong = false;
//OBV
if (ObvVal <= AvObv) SendLong = false;
//Market cross of chart trendline
//if (Bid <= ChartTrendVal) SendLong = false;
//if (Open[1] > PrevChartTrendVal) SendLong = false;
//Market must be above chart Moving Average
if (MaPeriod > 0 && Bid <= MaVal) SendLong = false;

/////////////////////////////////////////////////////////////////////////////////////

//Short trade
//Trendline colour
if (!TradeShort) SendShort = false;//Depends on trendline colour
//OBV
if (ObvVal >= AvObv) SendShort = false;
//Market cross of chart trendline
//if (Bid >= ChartTrendVal) SendShort = false;
//if (Open[1] < PrevChartTrendVal) SendShort = false;
//Market must be below chart Moving Average
if (MaPeriod > 0 && Bid >= MaVal) SendShort = false;

I have set the ea inputs for 50 SMA.

I have also commented out the idiot check for drawing chart trendline.

Are the modifications correct ?

I will post the results of my testing here.

Cheers,
Peter

[fx800]

Hi Steve

I am trying to modify your SAFT helper ea to test LDHT's settings (break of 50 SMA by price and break of 21 LWMA by OBV).

I have modified the buy/sell code as follows

//Long trade
//Trendline colour
if (!TradeLong) SendLong = false;
//OBV
if (ObvVal <= AvObv) SendLong = false;
//Market cross of chart trendline
//if (Bid <= ChartTrendVal) SendLong = false;
//if (Open[1] > PrevChartTrendVal) SendLong = false;
//Market must be above chart Moving Average
if (MaPeriod > 0 && Bid <= MaVal) SendLong = false;

/////////////////////////////////////////////////////////////////////////////////////

//Short trade
//Trendline colour
if (!TradeShort) SendShort = false;//Depends on trendline colour
//OBV
if (ObvVal >= AvObv) SendShort = false;
//Market cross of chart trendline
//if (Bid >= ChartTrendVal) SendShort = false;
//if (Open[1] < PrevChartTrendVal) SendShort = false;
//Market must be below chart Moving Average
if (MaPeriod > 0 && Bid >= MaVal) SendShort = false;

I have set the ea inputs for 50 SMA.

I have also commented out the idiot check for drawing chart trendline.

Are the modifications correct ?

I will post the results of my testing here.

Cheers,
Peter

The modified ea failed to trigger even though there were valid entries.

Obviously there are errors in the code.

Any help will be greatly appreciated.

I would undertake to post the results of forward testing if we have a working ea so that others may benefit as well if the ea is profitable.

It worked for LDHT on FF. We just need to verify that.

Peter

[mobthehop]

Hi Peter, great idea, I follow that thread over at FF as well...

It might be easier for the helpful members here, if you post your EA in mq4 format - devil is in the detail and all that....

Cheers

Mop

[fx800]

Hi Peter, great idea, I follow that thread over at FF as well...

It might be easier for the helpful members here, if you post your EA in mq4 format - devil is in the detail and all that....

Cheers

Mop

Hi Mop, no problem.

Mod. ea attached.

Peter

[fx800]

An EU buy. Fractal SL = 18.

Market waiting for Ben Bernanke speech from Jackson Hole today. Until then, expect sideways trading.

[Anonymous]

A GU buy, fractal SL = 12.

[fx800]

An EU buy. Fractal SL = 18.

Market waiting for Ben Bernanke speech from Jackson Hole today. Until then, expect sideways trading.

SL Moved to BE + 3.

[Anonymous]

A GU buy, fractal SL = 12.

SL Moved to BE + 3.

[fx800]

An EU buy. Fractal SL = 18.

Market waiting for Ben Bernanke speech from Jackson Hole today. Until then, expect sideways trading.

SL Moved to BE + 3.

Target TP of 36 pips hit. Against my expectations !

[fx800]

A GU buy, fractal SL = 12.

SL Moved to BE + 3.

Target TP of 24 pips hit.