Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

Documentation Typo

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

    Documentation Typo

    I belive the 6.5 beta has a documentaion typo regarding the OnOrderUpdate() method.

    The original text reads:

    As an example, the NinjaTrader core may have received "Working" and then "Part Filled" order state change events back from the broker API on thread "B" and at some point in time (milliseconds later) the NinjaTrader core will take these events and trigger the OnOrderUpdate() method in the strategy on thread "A". Thus, when the strategy receives the first "Working" state for an order, the IOrder object passed in will reflect the "Working" state although the actual order is really in a state of "Part Filled" which is truly reflected in the original IOrder object returned in any of the order methods such as EnterLong(). Of course, the OnOrderUpdate() method will subsequently receive the event for "Working" state.
    I believe it should be:

    As an example, the NinjaTrader core may have received "Working" and then "Part Filled" order state change events back from the broker API on thread "B" and at some point in time (milliseconds later) the NinjaTrader core will take these events and trigger the OnOrderUpdate() method in the strategy on thread "A". Thus, when the strategy receives the first "Working" state for an order, the IOrder object passed in will reflect the "Working" state although the actual order is really in a state of "Part Filled" which is truly reflected in the original IOrder object returned in any of the order methods such as EnterLong(). Of course, the OnOrderUpdate() method will subsequently receive the event for "Part Filled" state.
    It's not a big deal, but I thought I would mention it.

    Thanks,

    -Scott

    #2
    Thanks for bringing this to our attention.
    Josh P.NinjaTrader Customer Service

    Comment

    Latest Posts

    Collapse

    Topics Statistics Last Post
    Started by llanqui, 04-28-2024, 10:32 AM
    3 responses
    23 views
    0 likes
    Last Post bltdavid  
    Started by WHICKED, 04-26-2024, 12:56 PM
    5 responses
    152 views
    0 likes
    Last Post NinjaTrader_Gaby  
    Started by PaulMohn, Yesterday, 02:06 AM
    7 responses
    20 views
    0 likes
    Last Post PaulMohn  
    Started by f.saeidi, Today, 08:01 AM
    0 responses
    3 views
    0 likes
    Last Post f.saeidi  
    Started by ezrollin, 02-26-2022, 11:14 PM
    8 responses
    210 views
    0 likes
    Last Post NinjaTrader_ChelseaB  
    Working...
    X