Hmmmmm. As I glanced through the code, I saw a few bits and bats that looked remarkably like stuff from my ea's.dlewisfl wrote:A few other changes that can (or should) be made.SteveHopwood wrote:V 1a is in post one with this fix. Nice one, dlewisfl.
- The "BrokerIsECN" user input does nothing, there is no code for it. I deleted it on mine.
- The "Slippage" user input does nothing, there's no code for it. The value of 6 is hard-coded for slippage in all of the OrderSend commands. I changed each 6 to be the Slippage user input.
- The color for the "PivotLine" object is hard-coded as "Black" in the code, doesn't work well for those who use black background.I created a user-input for it on my version.
Looks like A Cowboy decided to pinch my stuff and failed to use it properly. Hey ho. All I can do is provide the original and hope that Cowboys are not too thick to do anything useful with it. Seems like a wasted hope here.
Guys, understand that my interest in what happens here is limited. I do not know what this ea does or how it trades - I have yet to add it to a demo, although I shall do so tomorrow. Let's just say that I am not hopeful of good results. So, it follows as night follows day that I am not going to try to keep up with umpteen revisions contributed haphazerdly.
So, dlewisfl, are you able to adapt the code inthe modified version in post 1 and add the relevant update history and update the version number and post it here? If so, fantastic and thanks; I will make it the post 1 version. When coders add to my code in other ea's or by pm, I add an appreciation of their work to the commentary. This is not my bot, so please add your own, "I did this" sort of thingy. "//Code added/deleted/adapted by dlewisfl who is also doing his best to drag this Abortion of code into the 20th century" sort of thing. 20th century was not a typo.