ForexPippy breakout EA

[gaheitman]

hmmm, a good example of where a trade ought to have been taken (nothing happened).

Box is drawn correctly, lovely breakout, just missing one minor element

I'm using the current version.

I'll do my best to have a look at the code tonight, although I'm still in the early stages of learning coding

1.gif

The ReadIndicatorValues() procedure still has the code to look at the lines drawn by the original indicator, and unfortunately that code is overriding the internal code to get the box high/low. Go ahead and delete the following lines from ReadIndicatorValues() (or, put the indicator on your chart ).

Code: Select all

   BoxHigh = ObjectGetValueByShift("DLB#1-0-hi", 0);
   BoxLow = ObjectGetValueByShift("DLB#1-0-lo", 0);

If you notice on your chart, in the comment, BoxHigh and BoxLow are both 0, which is what clued me in.

George

[SteveHopwood]

Fix for everyone else in the post 1 update.

[DING]

I am testing the lastest version , but the box seems have problem,using Alpari UK Empty4(GMT+1), set box hour from 3 to 7, But the drawing magenta box in my picture only calculate first hour candle,the Aqua box is the original Daily_LinesBoxes_OC indicator .what is wrong for me :

[SteveHopwood]

I am testing the lastest version , but the box seems have problem,using Alpari UK Empty4(GMT+1), set box hour from 3 to 7, But the drawing magenta box in my picture only calculate first hour candle,the Aqua box is the original Daily_LinesBoxes_OC indicator .what is wrong for me :

When I looked at my demo this morning, I noticed that some of the lines were not drawing properly.

As soon as I deleted the upper line (always do this; never mind why, just do it) the ea would redraw the lines correctly.

I am going to add code to delete the box once trading hours are passed in time for next week. Probably a bit of a brutal solution, but if it works, then fine.

[sahars]

Hi Steve,

Thanks a lot for opening this site for us to share our forex experience... BTW...I've coded (sort-of... ) the EA that follow the rules as per ForexPippy rule... (except this EA does not use 3/8EMA)..My code it was actually from your old code and I just copy and paste it become a complete code...So I believe some of the coding might be not right in this EA... So..please improved it as you wish if you think so of the code need to be improved...

I attached here the code and the backtest result... So far for 2 days forward test demo with EU, GU and UCHF, the EA is gaining more than 10%...

So.. happy testing and you are welcome to ask about this EA..

Cheers

sahars

[sahars]

Guys,

Sorry.. I forgot to attach the EA...

Cheers

sahars

[Xfinity]

Looks nice sahars! Any good source for LibOrderReliable?

[sahars]

Looks nice sahars! Any good source for LibOrderReliable?

Hi Xfinity,

My initial of this EA is using LibOrderReliable...but I decided to remove it since I do not want to confuse others on the installation procedure of this EA. Furthermore I don't see any benefits of using it...

BTW.. I attached the LibOrderReliable and the EA that not require this LibOrderReliable function.

So guys, please ignore the previous version and can use the attached version...

Cheers

sahars

[Xfinity]

Thanks Sahars! I will test it asap on my demo account.

[sahars]

Thanks Sahars! I will test it asap on my demo account.

Xfinity...

Since the time for calculating the box zone is already pass, you need to key-in manually by pressing F3 and key-in the High and Low Zone value manually... Otherwise... the EA will not work for today...

Sahars