From bad7e02e53922217e830a462fc1e9e2e53c223df Mon Sep 17 00:00:00 2001 From: Celes Renata Date: Thu, 16 Apr 2026 00:16:32 +0000 Subject: [PATCH] fix: add first-skip-reason logging to backtest replay for debugging --- services/trading/backtest_replay.py | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/services/trading/backtest_replay.py b/services/trading/backtest_replay.py index 8170149..59ea6bf 100644 --- a/services/trading/backtest_replay.py +++ b/services/trading/backtest_replay.py @@ -190,6 +190,17 @@ class BacktestReplay: } portfolio_state.active_pool -= cost portfolio_state.open_position_count += 1 + elif act_count == 0 and not hasattr(self, '_first_skip_logged'): + # Log the first skip reason for debugging + logger.warning( + "Backtest first skip: ticker=%s reason=%s conf=%.2f price=%.2f pool=%.2f", + decision.ticker, + decision.skip_reason, + rec.get("confidence", 0.0), + rec.get("current_price", 0.0), + portfolio_state.active_pool, + ) + self._first_skip_logged = True if day_recs: logger.info(