Drift rebalancer 5

[brettelliot]

This PR does two things (sorry - i found a bug and just fixed it).

1. Refactors the DriftRebalancer stuff into components
2. Fixes a bug in tradier_data. The recent changes i made to make tradier's get_historical_bars return the last N bars had an issue if you called it before market open. It would get one less bar then you asked for before market hours because it providded the last N bars from today... but before market hours there was no data for today. This is addressed now, and if you ask for data before market hours it will get you the last N bars excluding today. And it you ask for data during market hours, it will include todays bar in the N bars.

Description by Korbit AI

What change is being made?

Introduce a robust drift rebalancing framework with new components: DriftCalculationLogic, DriftRebalancerLogicBase, and LimitOrderDriftRebalancerLogic, and integrate these into a refactored DriftRebalancer strategy.

Why are these changes being made?

The changes enhance the drift rebalancing strategy by modularizing logic into distinct components for improved readability, maintainability, and testing. This refactoring addresses the need for a flexible and efficient approach to maintain target asset allocations, accounting for acceptable slippage and drift thresholds, and supports both rebalancing actions and limit order executions. The new framework also facilitates better testing and validation of each component's functionality, as evidenced by the added tests.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

[korbit-ai]

Korbit doesn't automatically review large (500+ lines changed) pull requests such as this one. If you want me to review anyway, use /korbit-review.