SteveHopwood wrote:Ehup guys

Just starting to catch up after 4 extremely frustrating days without an internet connection.

Re the possible missing bars thingy, the version in post 1 that I uploaded last Thursday includes the iBars thingy. In int init()

Code: Select all

//This forces the platform to 2048 bars of data - necessary if this is a new Empty4 installation
   iBars(NULL, Period() );

This should do the trick unless I have misunderstood George's post about this.

iBars(NULL,RsiTdTf);

gaheitman wrote:

SteveHopwood wrote:Ehup guys

Just starting to catch up after 4 extremely frustrating days without an internet connection.

Re the possible missing bars thingy, the version in post 1 that I uploaded last Thursday includes the iBars thingy. In int init()

Code: Select all

//This forces the platform to 2048 bars of data - necessary if this is a new Empty4 installation
   iBars(NULL, Period() );

This should do the trick unless I have misunderstood George's post about this.

You'll actually need to call it for any time frame you are referencing. The user may have it on an hourly chart but using the daily RSI, so you'd want to call (in this care):

Code: Select all

iBars(NULL,RsiTdTf);

George

MichaelM wrote:Another one in GetNextRoundNumbers():
...

Code: Select all

   double price = Bid;

MichaelM wrote:Hi Steve,

I've noticed today that quite a few of the pairs I've put Graeme on don't have the Pending Price Line drawn on the chart even though the Overall Trend says either "Up" or "Down".

Looked in the Objects list on each chart and there's nothing there.

Using Graeme OOTB

EDIT: I figured it out... in LookForTradingOpportunities() it uses the RsiBuffer whereas in the TrendDetectionModule it does not.

So you can either set the RsiBuffer = 0 or fix the TrendDetectionModule code as follows:

Code: Select all

//Rsi. This one is scooby-doo's suggestion and is based on a 20 period D1 Rsi.
   if (UseRsiTrendDetection)
   {
      RsiTrend = ranging;
      TrendRsiVal = GetRsi(RsiTdTf, RsiTdPeriod, RsiTdAppliedPrice, 0);
      if (TrendRsiVal > 55 + RsiBuffer) RsiTrend = up;
      if (TrendRsiVal < 45 - RsiBuffer) RsiTrend = down;
      if (RsiIsMaster) OverallTrend = RsiTrend;
   }//if (UseRsiTrendDetection)

MichaelM wrote:Another coding error in LookForTradingOpportunities():

Code: Select all

   //Short
   if (OverallTrend == down)
   {
      if (!TradeShort) return;

      //Big number filter
      if (RoundNumberHigh == BigNumberHigh && !TradeBigNumbers) return;

...should be...

Code: Select all

   //Short
   if (OverallTrend == down)
   {
      if (!TradeShort) return;

      //Big number filter
      if (RoundNumberLow == BigNumberLow && !TradeBigNumbers) return;