Files
foxhunt/ml/examples
jgrusewski aaaaef7f48 fix(ml): address code review — greedy PPO eval, gamma alignment, Sharpe fix
Fixes from code review of DQN/PPO validation pipeline:

1. PPO validation: use greedy_action() (argmax) instead of stochastic
   act() — deterministic early-stopping signal, matching DQN's eps=0.

2. DQN eval gamma: align to 0.95 (was 0.99) matching train_baseline.
   Gamma doesn't affect greedy inference but configs should match.

3. Sharpe annualization: use 1380 bars/day (23h futures session) not
   390 (6.5h equities). Fixes ~1.8x underestimate for ES futures.

4. compute_reward: accept f64 total_cost_bps (was f32) to match
   shared spread_cost_bps() from baseline_common.rs.

5. Default max_steps_per_epoch: 2000 (was 0/unlimited) for OOM safety.

6. Hyperopt PPO: add timestamp dedup to decode_ohlcv_bars for .FUT
   parent symbols with overlapping contracts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-24 01:02:51 +01:00
..