Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

OnExecutionUpdate()? Executions do not get updated!

Collapse
X
 
  • Filter
  • Time
  • Show
Clear All
new posts

    OnExecutionUpdate()? Executions do not get updated!

    That makes no semantic sense. Orders are updated; so are positions, so OnOrderUpdate() and OnPositionUpdate() make sense. Executions occur; they are never updated. Once an execution occurs, it is done.

    That being said, I am sure it may be too late to make a change back to OnExecution(). Hey, I am just saying.

    #2
    Internally, from the adapter/connection, there are 3 operations on an Execution: Add/Remove/Update (e.g, Add operation, we have to pair an order object to an execution). We expose the Update operation to NS as this is relevant to an end user.

    The name is accurate internally and since we're now giving you guys the actual internal events, the name makes sense from that perspective.
    MatthewNinjaTrader Product Management

    Comment


      #3
      Originally posted by NinjaTrader_Matthew View Post
      Internally, from the adapter/connection, there are 3 operations on an Execution: Add/Remove/Update (e.g, Add operation, we have to pair an order object to an execution). We expose the Update operation to NS as this is relevant to an end user.

      The name is accurate internally and since we're now giving you guys the actual internal events, the name makes sense from that perspective.
      Thanks for the response. Your argument makes sense from a development perspective.

      Comment

      Latest Posts

      Collapse

      Topics Statistics Last Post
      Started by ScottWalsh, Today, 06:52 PM
      3 responses
      17 views
      0 likes
      Last Post NinjaTrader_Manfred  
      Started by trilliantrader, Today, 03:01 PM
      2 responses
      17 views
      0 likes
      Last Post helpwanted  
      Started by cre8able, Today, 07:24 PM
      0 responses
      1 view
      0 likes
      Last Post cre8able  
      Started by Haiasi, Today, 06:53 PM
      1 response
      4 views
      0 likes
      Last Post NinjaTrader_Manfred  
      Started by ScottW, Today, 06:09 PM
      1 response
      6 views
      0 likes
      Last Post NinjaTrader_Manfred  
      Working...
      X