AtmStrategyChangeStopTarget() not working

[monpere]

AtmStrategyChangeStopTarget() changes the stop but not the target. The following code tries to continuously move both the stop and the target away from the market, but only the stop is being moved.

Create a strategy with the code below. Create and ATM called 'TestATM', with 1 contract, Stop Loss 50, Profit Target 50. Bring up the ER2 on a chart with Chart Trader. Connect to the simulator. Create and start the strategy below in control center for ER2 using 30 second bars, and watch the trade on the chart trader, only the stop moves, the target does not.

#region Variables
string tradeId="Trade";
string atm = "TestATM";
double price=0;
bool ret;
#endregion

protected override void Initialize() {
CalculateOnBarClose = true;
}

protected override void OnBarUpdate() {
if ( Historical ) return;

//### Create ATM
if (GetAtmStrategyMarketPosition(tradeId) == MarketPosition.Flat) {
tradeId = "Trade"+CurrentBar;
if (!AtmStrategyCreate(Action.Buy, OrderType.Market, Close[0], 0, TimeInForce.Day, tradeId, atm, tradeId))
Print ("*** Error creating " +tradeId);
}

//### Move Target
price = Close[0]+(40*TickSize);
ret = AtmStrategyChangeStopTarget(0,price, "TARGET1", tradeId);
if ( ret == true )
Print("Moved target to " +price.ToString("#.##"));
else Print("*** Failed to move target");

//### Move Stop
price = Close[0]-(40*TickSize);
ret = AtmStrategyChangeStopTarget(0,price, "STOP1", tradeId);
if ( ret == true )
Print("Moved stop to " +price.ToString("#.##"));
else Print("*** Failed to move target");
}

[NinjaTrader_JoshP]

Are you getting any errors in Control Center logs along with this? I will play with this in the meantime.

[monpere]

No errors in the error log. Not, only that, the return value of the method is 'true'.

Are you getting any errors in Control Center logs along with this? I will play with this in the meantime.

[monpere]

Any news on this issue?

No errors in the error log. Not, only that, the return value of the method is 'true'.

[MJT]

Is this the cause?
Your line
ret = AtmStrategyChangeStopTarget(0,price, "TARGET1", tradeId);
should be
ret = AtmStrategyChangeStopTarget(price, 0, "TARGET1", tradeId);

[monpere]

You have hit the nail on the head MJT. That is the problem. I guess because you provide a name for the order as one of the parameters, I neglected to look at the prices, because since you can identify the order by the name, a separate parameter for target and stop price is redundant.

Thanks. Good catch!

Is this the cause?
Your line
ret = AtmStrategyChangeStopTarget(0,price, "TARGET1", tradeId);
should be
ret = AtmStrategyChangeStopTarget(price, 0, "TARGET1", tradeId);

[NinjaTrader_Ray]

Yes, thanks MJT!

[NinjaTrader_JoshP]

Thanks MJT. I'm going blind here on my end and was just staring at it for awhile.