Position not being closed in strategy analyzer

[NinjaTrader_ChelseaB]

Hello tkaboris,

Can you print the time and the high and low of each bar to confirm the price was hit while the position was open and this order was working?

That should be the first line in OnOrderUpdate(). No this would be placed after an order.

https://ninjatrader.com/support/helpGuides/nt8/onorderupdate.htm

[tkaboris]

i am confused about print order line. Is it after onbarupdate or after i place an entry

[NinjaTrader_ChelseaB]

Hello tkaboris,

You have placed this in OnBarUpdate(). You need to place this in OnOrderUpdate().

The order object is a parameter of the OnOrderUpdate() method call.

protected override void OnOrderUpdate(Order order, double limitPrice, double stopPrice, int quantity, int filled, double averageFillPrice, OrderState orderState, DateTime time, ErrorCode error, string comment)

[tkaboris]

ok i printed as directed

https://drive.google.com/file/d/15ts37_dJp4-huT20XLQJ8Tq5zi-kgYTM/view?usp=sharing

[tkaboris]

i posted my code in #15... can you help to place this line correctly?
protected override void OnOrderUpdate(Order order, double limitPrice, double stopPrice, int quantity, int filled, double averageFillPrice, OrderState orderState, DateTime time, ErrorCode error, string comment)

[NinjaTrader_ChelseaB]

Hello tkaboris,

Copy and paste the order method call from the Syntax section in the help guide.

Place this is the scope of the class. (For example below OnBarUpdate())

[tkaboris]

I tried putting this line everywhere but I get errors. thats why i asked if you can help

[NinjaTrader_ChelseaB]

Hello tkaboris,

This is again placed in the scope of OnBarUpdate(). This not supposed be in the scope of OnBarUpdate(). OnBarUpdate(), OnStateChange() are methods that are in the scope of the class.

Try placing the entire method declaration below the action block of OnBarUpdate().

If you are unfamiliar with scope in C#, I am referring to the curly braces. Anything between a set of curly braces is inside the scope of that method or class. Place this below the closing curly brace of OnBarUpdate().

Below is a link to an example script that has the declaration for OnOrderUpdate() implemented.

https://ninjatrader.com/support/forum/forum/ninjatrader-8/platform-technical-support-aa/97419-priceos-cs-expecting?p=802269#post802269

[tkaboris]

Ok I was able to include that override method and i wanted to copy wahts in the output
i am getting
ou have reached the maximum threshold of the NinjaScript Output window. and window is blank now

[tkaboris]

here is a file
https://drive.google.com/file/d/15ts...ew?usp=sharing

chart looks like this


Please help

[NinjaTrader_ChelseaB]

Hello tkaboris,

I'm still not seeing the print of the order object.. Have you compiled, cleared the output window, and re-run the script after making the changes?

Also, I am not seeing in the output where the high rose to 15250.5.
6/13/2023 4:59:59 PM Strategy 'Stoch Algo Trader/-1': Amended parabolic stop order orderId='NT-00006-14208' account='Backtest' name='Parabolic stop' orderState=Working instrument='MNQ 09-23' orderAction=BuyToCover orderType='Stop Market' limitPrice=0 stopPrice=15250.5 quantity=1 tif=Gtc oco='NT-00004-14208' filled=0 averageFillPrice=0 onBehalfOf='' id=-1 time='2023-06-12 19:05:25' gtd='2099-12-01' statementDate='2023-07-07' stopPriceChanged=15250.5

As we verified SetParabolicStop is being set and amended, you can temporarily disable TraceOrders to reduce the number of prints.

[tkaboris]

I disabled trace orders but still getting lots of prints. Should I comment out this line? Print(order.ToString());
i did recompiled it several times... and ran several times after that..

[tkaboris]

strategy just in case

[NinjaTrader_ChelseaB]

Hello tkaboris,

This is the output we want to see. Along with the high and low of the bar to verify that price crossed above the stop price of the working parabolic stop order.

At what time does the market price rise past 15250?

Be sure to save the output to a text file and include this with your post.

[tkaboris]

I am lost. I understand what you want to see but these prints with orderid flooding the output...