From ed598888a9e22a7fc378e9079ffac9b0fc5fe88f Mon Sep 17 00:00:00 2001 From: jgrusewski Date: Thu, 13 Nov 2025 21:02:26 +0100 Subject: [PATCH] Fix unused variable warning in portfolio_integration_tests.rs Wave 16S-V14: Code quality improvement Changes: - Prefixed unused variable _features_after_buy with underscore - Eliminates warning: unused variable 'features_after_buy' at line 364 - No functional changes, purely cosmetic fix Impact: 0/0 warnings in ml crate (100% clean) Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- ml/src/dqn/tests/portfolio_integration_tests.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ml/src/dqn/tests/portfolio_integration_tests.rs b/ml/src/dqn/tests/portfolio_integration_tests.rs index 6ad482207..0610ac6c9 100644 --- a/ml/src/dqn/tests/portfolio_integration_tests.rs +++ b/ml/src/dqn/tests/portfolio_integration_tests.rs @@ -361,7 +361,7 @@ fn test_portfolio_tracking_hold_action() -> anyhow::Result<()> { // Execute BUY to create a position tracker.execute_action(buy_action(), 100.0, 10.0); - let features_after_buy = tracker.get_raw_portfolio_features(100.0); + let _features_after_buy = tracker.get_raw_portfolio_features(100.0); // Execute HOLD action (Flat exposure = close position) // Note: In FactoredAction system, HOLD means "target Flat exposure" = close all positions