mirror of
https://github.com/microsoft/qlib.git
synced 2026-07-09 22:10:56 +08:00
Fix account update bar_count bug
This commit is contained in:
@@ -263,11 +263,11 @@ class Account:
|
|||||||
elif atomic is False and inner_order_indicators is None:
|
elif atomic is False and inner_order_indicators is None:
|
||||||
raise ValueError("inner_order_indicators is necessary in unatomic executor")
|
raise ValueError("inner_order_indicators is necessary in unatomic executor")
|
||||||
|
|
||||||
|
# TODO: `update_bar_count` and `update_current` should placed in Position and be merged.
|
||||||
|
self.update_bar_count()
|
||||||
|
self.update_current(trade_start_time, trade_end_time, trade_exchange)
|
||||||
if generate_report:
|
if generate_report:
|
||||||
# report is portfolio related analysis
|
# report is portfolio related analysis
|
||||||
# TODO: `update_bar_count` and `update_current` should placed in Position and be merged.
|
|
||||||
self.update_bar_count()
|
|
||||||
self.update_current(trade_start_time, trade_end_time, trade_exchange)
|
|
||||||
self.update_report(trade_start_time, trade_end_time)
|
self.update_report(trade_start_time, trade_end_time)
|
||||||
|
|
||||||
# indicator is trading (e.g. high-frequency order execution) related analysis
|
# indicator is trading (e.g. high-frequency order execution) related analysis
|
||||||
|
|||||||
Reference in New Issue
Block a user