GetProfitLoss for manual trades

[ATI user]

in MR

if I make an atmstrat trade from my strat, I can pick up unrealized and realized (and totalpnl when NT adds it asap)

if I make a non-atm trade from a strat, I can pick up unrealized with GetProfitLoss

if I make a manual trade, I can not pick up unrealized from code. Correct?

If correct, why not....i.e. can you add asap?

If incorrrect, how?

Thanks.

[NinjaTrader_JoshP]

This is not allowed. Strategies are not meant to interact with any manual trades or any trades from other strategies not part of it.

[ATI user]

post edited to another thread

[NinjaTrader_JoshP]

Incorrect. Strategies is not supported to have access to information of trades done manually. Each strategy is only aware of the trades it makes itself. It does not recognize anything done anywhere else.

[ATI user]

Request - Global Risk Management

post edited to another thread

[NinjaTrader_JoshP]

- using non-AtmStrategyCreate code users have access to Realized PnL only. Fine if it is a small position held for a short period of time...i.e. the unrealized PnL is negligible. When unrealized becomes 10 or 20 times the realized, then realized is useless....especially if unrealized is a large loss and you want to exit automatically for the day.

Position.GetProfitLoss() gives you the unrealized PnL.

1. Not true. Though the MA may show you everything in relation to your account, strategy code is only for the strategy. This is why there is the whole thing about strategy position versus account position.

2. Correct. Position.GetProfitLoss() gets you the unrealized PnL for the NinjaScript managed position.

3. Instead of flooding the language with unnecessary methods, especially for something that just requires something as trivial as adding two double values together, we find it is better done this way.

Please note that your understanding of Position.GetProfitLoss() = realized PnL is inaccurate. It gives you the unrealized PnL of your strategy position.

I have added your request for unrealized PnL for the account position to our list of considerations.

[ATI user]

post edited to another thread

[ATI user]

Please note that your understanding of Position.GetProfitLoss() = realized PnL is inaccurate. It gives you the unrealized PnL of your strategy position.

yes ...my typo...have corrected above...point being we need both

[ATI user]

I have added your request for unrealized PnL for the account position to our list of considerations.

hope you mean you added Realized to list?

[NinjaTrader_JoshP]

Not following you. From a NinjaScript strategy you have access to everything in terms of PnL for the strategy. You have realized and unrealized. For realized, just go through the Performance class to grab it out. For unrealized, use GetProfitLoss().

Hedging makes no difference. Each ATM strategy you create is its own entity with its own realized/unrealized with GetAtmStrategyRealizedProfitLoss() and GetAtmStrategyUnrealizedProfitLoss(). If you are hedging with multiple strategies just run the method on the multiple ATM strategy IDs.

I.e. ATM strategy A = long; ATM strategy B = short.
Total PnL of A and B = GetAtmStrategyRealizedProfitLoss(strategyA) + GetAtmStrategyUnrealizedProfitLoss(strategyA) + GetAtmStrategyRealizedProfitLoss(strategyB) + GetAtmStrategyUnrealizedProfitLoss(strategyB)

This is very basic arithmetic that does not warrant providing an extra method. If you really want this you can just as easily make your own method. You have the components to calculate what you want easily accessible.

[ATI user]

post edited to another thread

[psusen]

RE: GetProfitLoss for manual trades

I also see the need for having an 'all inclusive' risk management system. Like many traders I am sure, I have trouble from time-to-time with overtrading, but with something like this in place (an account risk management, essentially), whether auto-trading or manual trading, I would have something to 'remind' me to stop trading, and that stops the trading based on certain risk parameters. Being able to contol risk at this level would be beneficial to all I believe.

My 2 cents,
Peter

[ATI user]

post edited to another thread

[NinjaTrader_JoshP]

As mentioned, this is not supported and has already been added to our feedback list.