Any help ? struggle to code this price channel in NT8

[Lyubomir Kratinov]

I'm tithing 2 weeks to code this in NT8 , but no result yet , any help?

I can't adopt price series to get this done.
Thanks in advance

[NinjaTrader_ChrisL]

Hello Lyubomir Kratinov,

Thank you for the post, and welcome to the NinjaTrader forum.

I see you are looping through all of the bars on the chart. NinjaTrader will do this for you automatically when you load your indicator onto the cart, I call it the "historical run". All you have to do is put your logic in OnBarUpdate and NinjaTrader will run that logic for each bar of the chart (with OnBarClose). For an example of this look at the source code for a simple indicator like the ATR. All of the OnBarUpdate logic is processed when the indicator is loaded, so you see the historical values of ATR.

Please try moving the logic from within the FOR loop and place that within OnBarUpdate. Also replace any 'i' indexes with a '0' referencing the most current bar.

I can take a look at the full code as well if you would like.

I look forward to hearing of your results.

[Lyubomir Kratinov]

Here is the code : // but not works as well

[NinjaTrader_ChrisL]

Hello Lyubomir Kratinov,

Thank you for the reply.

I added a CurrentBars check at the beginning of OnBarUpdate and your plots worked after that. Remember to check the Log tab of your Control Center if you apply an indicator to your chart and receive no results CurrenBars counts upward from the leftmost bar of the chart, so the problem was you were accessing LineHigh[1] when the script only had bar 0 to work with.

protected override void OnBarUpdate()
{
if(CurrentBar < 1)
return;

//rest of the code.

Please let me know if I can assist further.

[NinjaTrader_ChrisL]

Hello Lyubomir,

Thanks for the reply.

Could you please post the NT7 code so I can take a look?

I look forward to your reply.

[seykool]

This indicator is awesome! I love it!
Though I've never coded C# and hardly code at all I accomplished it to add:

private double Deviation1;
Deviation1 = Deviation1 * Bars.Instrument.MasterInstrument.TickSize;

in order to be able to use the indicator on instruments like CL, ES or GC which don't have 1 tick as one 1.00 like YM for e.g.

After trading a while with it I recognized that it misses every 20th wick or so in its calculation. There seems to be no regularity. Just rarely it ignores a wick but it does it sometimes, unfortunately.
Lyubomir Kratinov did you already solved this or do you guys have an idea what might be the matter?

[caveat_lector]

Bar highs and lows are missed sometimes, because of basing the high and low price tests on close, instead of high and low. These two changes will fix the problem:

//            if (Close[0] > aaMax[1])
            if (High[0] > aaMax[1])
                aaTrend[0] = 1;

//            if (Close[0] < aaMin[1])
            if (Low[0] < aaMin[1])
                aaTrend[0] = -1;

[seykool]

Indeed that fixed it. Very nice. Thank you Lector!
Makes sense with the Close vs High/Low

[caveat_lector]

Even though the indicator works properly with those changes, it wastes lots of memory due to poor design. A much more efficient version is attached.

HLchannel.zip

[seykool]

Great.

I digged a little into the code and added an edge area by 2 additional lines and a trend indicating background.

[caveat_lector]

seykool, the file you posted above contains "Using declarations" and "NinjaScript generated code" regions, but no indicator code.

[seykool]

Awesome! I made the indicator in the end exactly like you have it, just with a transition background, and trade always this one system. I love it, it's a beauty! I am so happy somebody did the coding I couldn't do.
And finally I know how the system is actually called, haha.

[jvar6]

Hi LK, where can I download please?

Regards,
Jag