foreach BarsInProgress?

February 17th, 2012, 02:03 PM

jeremytang

Hi ShruggedAtlas,

In the code you've provided, the for loop is redundant... so you can get the exact same effect with it removed.

My question is do you:

1) Want to perform your code on ALL time frames all the time (ie: whenever ANY of the timeframes are updated)

or

2) Only perform your code for the timeframe that is currently being updated

If you fall into category 1, then you need a for loop, and you'll have to reference all the closing prices, highs and lows etc by: Closes[x][0], Highs[x][0] etc. (note the plural).

Otherwise, if you're category 2 you can omit the for loop and access your data using Close[0], High[0] (etc) <-- no plural.

OnBarUpdate is called for each BarsInProgress in turn, and the Close, High (etc) DataSeries are changed to reflect the current BarsInProgress.

I assume you're more likely category 2, because you are referencing SMA, EMA and regressionChannel on the main timeframe (BarsInProgres == 0), am I right? Or do you intend to check the regressionChannel on every timeframe?

Jeremy

Actually I need to reference the indicators for all time frames. What I ended up doing was getting rid of the for loop and accessing the other DataSeries only from within the primary bars update if(BarsInProgress == 0)

it works pretty well overall that way.

February 17th, 2012, 02:11 PM

@Fat Tails

thanks FT - You've given me some new info that help a lot. I realize now that i was causing a lot more work for my cpu than necessary. I got rid of the for loop and it works fine. It is still updating on each tick for every conditions on all time frames so I need to spend some time optimizing the algorithm so it doesn't update the longer time frames needlessly. this will go a long way to increasing the performance...still working on that aspect of it.

foreach BarsInProgress?

Discussion in NinjaTrader

foreach BarsInProgress?