Email for order pending/fill/target/stop

[m3power222]

Please try with all caps. "TARGET1"

Tried that and still does not work.

Looked through the log and see this:
Category Strategy in yellow 'TARGET1' does not exist

Category Order Name='Target1' New State=Working............

[NinjaTrader_JoshP]

Does your ATM strategy actually have a target order then?

[m3power222]

Does your ATM strategy actually have a target order then?

Here is what it looks like.

[NinjaTrader_JoshP]

Are you sure you are passing in the correct atmStrategyId for the ATM that is actually active?

[m3power222]

Are you sure you are passing in the correct atmStrategyId for the ATM that is actually active?

I have this:

private string atmStrategyId = string.Empty;

[NinjaTrader_JoshP]

You should be storing the ID of your atm strategy into that string. Please ensure the string is not empty when you try to do GetAtmStrategy...().

[m3power222]

You should be storing the ID of your atm strategy into that string. Please ensure the string is not empty when you try to do GetAtmStrategy...().

I have no idea, do you have an example?

[NinjaTrader_JoshP]

Please take a look at the SampleAtmStrategy.

if (orderId.Length == 0 && atmStrategyId.Length == 0 && Close[0] > Open[0])
{
atmStrategyId = GetAtmStrategyUniqueId();
orderId = GetAtmStrategyUniqueId();
AtmStrategyCreate(Cbi.Action.Buy, OrderType.Limit, Low[0], 0, TimeInForce.Day, orderId, "AtmStrategyTemplate", atmStrategyId);
}

[m3power222]

Please take a look at the SampleAtmStrategy.

if (orderId.Length == 0 && atmStrategyId.Length == 0 && Close[0] > Open[0])
{
atmStrategyId = GetAtmStrategyUniqueId();
orderId = GetAtmStrategyUniqueId();
AtmStrategyCreate(Cbi.Action.Buy, OrderType.Limit, Low[0], 0, TimeInForce.Day, orderId, "AtmStrategyTemplate", atmStrategyId);
}

Yes I have all that stuff as my strategy is modified version of the SampleAtmStrat.

Does atmStrategyId = GetAtmStrategyUniqueId(); need to be put somewhere else again.

[NinjaTrader_JoshP]

Please post complete code of what you have. Thank you.

[m3power222]

Please post complete code of what you have. Thank you.

Here you go:



// Check for a pending entry order
if (orderId.Length > 0)
{
string[] status = GetAtmStrategyEntryOrderStatus(orderId);

// If the status call can't find the order specified, the return array length will be zero otherwise it will hold elements
if (status.GetLength(0) > 0)
{
// Print out some information about the order to the output window
Print("The entry order average fill price is: " + status[0]);
Print("The entry order filled amount is: " + status[1]);
Print("The entry order order state is: " + status[2]);

// If the order state is terminal, reset the order id value
if (status[2] == "Filled" || status[2] == "Cancelled" || status[2] == "Rejected")
{
orderId = string.Empty;
SendMail("[email protected]", "[email protected]", "Order", status[2]);
}
}
} // If the strategy has terminated reset the strategy id
else if (atmStrategyId.Length > 0 && GetAtmStrategyMarketPosition(atmStrategyId) == Cbi.MarketPosition.Flat)
atmStrategyId = string.Empty;


if (atmStrategyId.Length > 0)
{
// Print some information about the strategy to the output window
Print("The current ATM Strategy market position is: " + GetAtmStrategyMarketPosition(atmStrategyId));
Print("The current ATM Strategy position quantity is: " + GetAtmStrategyPositionQuantity(atmStrategyId));
Print("The current ATM Strategy average price is: " + GetAtmStrategyPositionAveragePrice(atmStrategyId)) ;
Print("The current ATM Strategy Unrealized PnL is: " + GetAtmStrategyUnrealizedProfitLoss(atmStrategyId)) ;
}

// Check for a pending entry order
if (atmStrategyId.Length > 0)
{
string[,] orders = GetAtmStrategyStopTargetOrderStatus("Target1", atmStrategyId);

// Check length to ensure that returned array holds order information
if (orders.Length > 0)
{
for (int i = 0; i < orders.GetLength(0) - 1; i++)
{
Print("Average fill price is " + orders[i, 0].ToString());
Print("Filled amount is " + orders[i, 1].ToString());
Print("Current state is " + orders[i, 2].ToString());
SendMail("[email protected]", "[email protected]", "Target", orders[i, 2].ToString());
}
}
}

}

[NinjaTrader_JoshP]

Please comment out this line: atmStrategyId = string.Empty;

And then try again so as to ensure it is not empty. Please also use different variables for your two different orders. You should have a completely separate set of ID tracking variables for your longs and shorts.

[m3power222]

Please comment out this line: atmStrategyId = string.Empty;

And then try again so as to ensure it is not empty. Please also use different variables for your two different orders. You should have a completely separate set of ID tracking variables for your longs and shorts.

I've commented out the variable.

To see if it is not empty do I need to Print()?

The single ID tracking vars are working they way they are...???

[NinjaTrader_JoshP]

Right. You could do a print. The single ID tracking is good for tracking one at a time. I am not sure if your strategy would start crossing over each other or not. Easiest just to keep it clean and simple and use two sets.

[m3power222]

Right. You could do a print. The single ID tracking is good for tracking one at a time. I am not sure if your strategy would start crossing over each other or not. Easiest just to keep it clean and simple and use two sets.

When I compile I get a bunch of atmstrategyId's do not exist...