TradingHours missing function

[DaveE]

TradingHours has a function GetPreviousTradingDayEnd(DateTime timeLocal)
but it does not have a matching function say called
GetPreviousTradingDayStart(DateTime timeLocal)
To be honest a name like GetSessionStart(DateTime timeLocal) would be better, but GetPreviousTradingDayStart matches the existing naming convention.

I know someone is going to say "use SessionIterator instead", but I want to be able to get the start of a session relative to a given DateTime in OnBarUpdate(), but as of RC2 the documentation for SessionIterator.GetNextSession() says

calling this method for each bar in the OnBarUpdate() method would NOT be recommended.

There is no such warning for TradingHours.GetPreviousTradingDayEnd() so I am assuming it is more efficient (and so the matching GetPreviousTradingDayStart should be also).

So could NT add GetPreviousTradingDayStart(DateTime timeLocal)
please.

[NinjaTrader_PatrickH]

Hello DaveE,

Thank you for your post.

You could use Bars.BarsSinceNewTradingDay to calculate the bars back for the start of the session. Then pass this to Time.

Bars: http://ninjatrader.com/support/helpG...en-us/bars.htm

[DaveE]

BarsSinceNewTradingDay only works in the context of the current session, it does not allow finding the session start for an arbitrary DateTime, so it cannot substitute for the suggested function GetPreviousTradingDayStart(DateTime timeLocal)

[NinjaTrader_Matthew]

SessionIterator is the ultimate core object used to traverse trading hours details over segments of time.

GetPreviousTradingDayEnd() is no more efficient than using your own session iterator.

Any helper methods we would create would simply wrap a session iterator object, and would need to call either sessionIterator.CalculateTradingDay or sessionIterator.GetNextSessionEnd to arrive at this information requested.

We will submit your request to add this type of method to the TradingHours class, but in the meantime - go ahead and use your own session iterator.

I also have requested an update to the help guide to reflect a similar performance concern

[NinjaTrader_PatrickH]

Hello DaveE,

Thank you for your patience.

This feature request has been assigned the ID SFT-1730.