BarsRequired error...maybe??

[funk101]

Trying to call an indicator from my strategy:

private KeltnerEmaAtr k;

protected override void OnStartUp()
{
	k = new KeltnerEmaAtr();
}
~~~~~~

protected override void OnBarUpdate()
{
        if (CurrentBar < k.MainPeriod) // (k.MainPeriod = 20)
		return;
	Print (Close[0]+" = "+k.Upper[0]);
}

I thought testing for CurrentBar/k.MainPeriod was the correct thing to do, instead I keep getting this message:

**NT** Error on calling 'OnBarUpdate' method for strategy 'AutoSysTrader/d2d5700a1a99406b9073e5b5a3922b3a': Index was outside the bounds of the array.

What am I missing here?

[NinjaTrader_CalH]

Hello Funky101.

Thank you for your note.

Try spreading out the prints with this matter.

Example -
Print(Close[0]);
Print(k.Upper[0]);

Also, is MainPeriod a public or private variable that you are trying to get for the minimum bars value?

[funk101]

Yeah, it Prints Close[0] but fails on k.Upper[0];

MainPeriod is a public var seen below

[Description("Numbers of bars used for calculations")]
[GridCategory("Parameters")]
public int MainPeriod
{
	get { return mainPeriod; }
	set { mainPeriod = Math.Max(1, value); }
}

[funk101]

By the way..

It fails when I use the regular KeltnerChannel() too

[NinjaTrader_CalH]

Funk101,

Can you send me your script including what you have created for the KeltnerEma?

[funk101]

What address?

[NinjaTrader_CalH]

Funk101,

Apologies, I jumped the post button too soon.

Please send these to platformsupport [at] ninjatrader [dot] com with ATTN Cal in the subject and reference this thread in the body

[funk101]

Sent. Again, this will fail with the regular KeltnerChannel() as well. It does however work with, for example, CCI()

[NinjaTrader_CalH]

Funk101,

The reason for this is because you are trying to create a new instance and not the indicator object.

You need to use the Indicator class with the overload parameters as such -

k = KeltnerEmaAtr(false, 20, 10, 2.0, false);


Let me know if I can be of further assistance.

[funk101]

Thanks, that was the answer *without* "new"