Check if chart is Minute based

[swcooke]

I am developing an indicator that requires minute bars and no other periodicity will work. How and where can I check that? I see that in the BuySellPressure Indicator example, it is being checked in State.DataLoaded. However this is not working for me:

if (State == State.DataLoaded)
        if (BarsPeriodType == BarsPeriodType.Minute)
        {
            Draw.TextFixed(this, "tag1", "Chart must be a minute based chart.", TextPosition.BottomRight); 
        }
}

My two questions are:

a.) is this the right place to check and
b.) is there some other way since this is returning an error: 'NinjaTrader.Data.BarsPeriodType' is a 'type' but is used like a 'variable'

[NinjaTrader_Jesse]

Hello swcooke,

Thank you for the post.

I wanted to check, where did you find this syntax specifically? I do not see this in the BuySellPressure indicator but I do see this is used in the ImportTypes which would make sense as to why you are getting this error. This is not a variable an Indicator would inherit so this would not be able to be used like shown from an indicator.

You could use something like the following:

Print(Bars.BarsPeriod.BarsPeriodTypeName);

This would print "Minute" for a Minute type bar series. This should also be fine to use from DataLoaded however Historical or Realtime could also be used here. You may additionally look at the BarTimer and its check:

if (Bars.BarsType.IsTimeBased && Bars.BarsType.IsIntraday)

I look forward to being of further assistance.

[swcooke]

Hi Jesse,

I should have mentioned that I was referring to the concept of displaying the text rather than the actual condition. The condition in BuySellPressure is checking to make sure that Calculate == Calculate.OnEachTick. Can you confirm that State.DataLoaded is the recommended place to check? In thinking about this further, should I just check right inside OnBarUpdate and return from OnBarUpdate if the condition tests true? Any downsides to this?

[NinjaTrader_Jesse]

Hello swcooke,

If the point is to draw text, I would suggest moving this to OnBarUpdate like you noted and also return to prevent logic if that is the overall goal. That just reduces the number of times this logic is being used as it seems the overall use is going to target OnBarUpdate. I don't see any reason why this would have a downside being used from OnBarUpdate for this use case.


I look forward to being of further assistance.