Cloning NT8 TrendLines indicator -- need help

[vantojo]

Hello,

I want to clone the NT8 TrendLines indicator and make some changes to it...

After I save this indicator to a AATrendLines then try to compile it....I get these errors

---------------------------------------------------------------------------------------------------------------------------------------------------

The best overloaded method match for 'NinjaTrader.NinjaScript.Indicators.AATrendLines.T rendQueue.TrendQueue(NinjaTrader.NinjaScript.Indic ators.TrendLines, int)' has some invalid arguments

Argument 1: cannot convert from 'NinjaTrader.NinjaScript.Indicators.AATrendLines' to 'NinjaTrader.NinjaScript.Indicators.TrendLines'

---------------------------------------------------------------------------------------------------------------------------------------------------

on this statement in OnStateChanged

trendLines = new TrendQueue(this, NumberOfTrendLines);


It seems this is a scope issue with Public ?

But, I'm not sure how to fix it...

????

Thanks

[NinjaTrader_ChrisL]

Hello vantojo, thanks for your note.

TrendLines has a class in it called TrendQueue. Change the constructor of that class to this and the compile error will clear:

private class TrendQueue : Queue<TrendRay>
{

    ...
    public TrendQueue([U]AATrendLines[/U] instance, int capacity) : base(capacity)
            {
                this.instance = instance;
            }

}

Kind regards.

[vantojo]

Perfect!

Thanks

[vantojo]

Hello,

I thought I had this fixed but when I revisited the indicator, I get this message

AATrendLines.cs

Cannot implicitly convert type 'NinjaTrader.NinjaScript.Indicators.AATrendLines' to 'NinjaTrader.NinjaScript.Indicators.TrendLines'

I've attached the code...

Thank you

[-=Edge=-]

I thought I had this fixed but when I revisited the indicator, I get this message

Couple months ago now, I posted a modified version at the link below:

You might take a look at that, or even use it for your base.. Good Luck!

https://ninjatrader.com/support/forum/forum/ninjatrader-8/platform-technical-support-aa/1056532-automatic-trend-lines-indicator-feature-requests?p=1056709#post1056709

[vantojo]

Thanks, will test it soon....

[vantojo]

would still like to fix the one I posted above....I don't need to manually move the lines....

[NinjaTrader_ChrisL]

Hi vantojo, thanks for your reply.

The TrendQueue class has an instance property, change that to reflect your change to the indicator class.

private class TrendQueue : Queue<TrendRay>
        {
            // Old: private TrendLines    instance;
            private AATrendLines    instance;
...

Kind regards.

[vantojo]

ok, that compiled now....thank you Chris