PerformanceMetric debugging

[tolisss]

Hi

I am trying to debug the OnAddTrade method but it looks like it is called multiple times for the same trade. Is there a way to filter those calls just for debugging purposes?

[NinjaTrader_Jesse]

Hello,

Thank you for posting.

I wanted to collect some information regarding the current use. Can you tell me, is this currently being used in backtesting or was this from a historical/realtime trades?

Also, how many times per trade are you seeing this being called?

I look forward to being of further assistance.

[tolisss]

Hello,

Thank you for posting.

I wanted to collect some information regarding the current use. Can you tell me, is this currently being used in backtesting or was this from a historical/realtime trades?

Also, how many times per trade are you seeing this being called?

I look forward to being of further assistance.

roughly 5 times and is called from both historical/realtime and backtesting modes

[tolisss]

in addition the metric is instanciated multiple times so a private flag to filter does not work, a static does however u need to reload the whole app domain

[NinjaTrader_Jesse]

Hello,

Thank you for the reply.

I had to confirm with development on this item, after testing I do see that it is variable at how many times this override is called and this is expected. The reason this is expected is that this metric is being called for multiple collections where the trade is added.

A more accurate description proposed for the help guide definition of this method would be as follows:

This method is called as each trade is added to each performance collection, this means this is called multiple times per trade. This is a pure function used for your calculations, please keep all logic inside of the method body to retain as a pure function.

Using a performance metric with private variables or otherwise is not suggested as they are designed to be pure methods that simply do a calculation on the added trade. Being called multiple times should not affect your logic, if it currently does you would need to change the logic to be a pure function that does not rely on class variables.

I look forward to being of further assistance.