Skip to content

Commit 826211c

Browse files
committed
BUG: Fix trades reported in reverse chronological order when finalize_trades=True
Fixes #1144
1 parent 2a9ea3b commit 826211c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

backtesting/backtesting.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -1320,7 +1320,7 @@ def run(self, **kwargs) -> pd.Series:
13201320
else:
13211321
if self._finalize_trades is True:
13221322
# Close any remaining open trades so they produce some stats
1323-
for trade in broker.trades:
1323+
for trade in reversed(broker.trades):
13241324
trade.close()
13251325

13261326
# HACK: Re-run broker one last time to handle close orders placed in the last

0 commit comments

Comments
 (0)