gitadmin/foxhunt
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(services): add validation-mod ml feature to trading + backtesting

Browse Source 
ml::validation is gated behind validation-mod, but dqn/config.rs:177
references crate::validation::RegimeMetrics unconditionally. Both
services pull in dqn::config via the trainer pipeline, so they need
the gate opened to compile. Cheaper than refactoring 700+ refs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
jgrusewski
2026-05-02 18:20:59 +02:00
parent 96769d1711
commit f58d4890ff
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
services/backtesting_service/Cargo.toml
View File

@@ -72,7 +72,7 @@ clap = { version = "4.5", features = ["derive"] }
# path-form: cargo 1.89's `workspace = true` ignores `default-features = false`,
# so we use the explicit path declaration to opt out of `full-stack` and drop
# 8 leaf-level ml-* sub-crates that no source file in this service references.
ml = { path = "../../crates/ml", default-features = false, features = ["financial", "cuda"] }
ml = { path = "../../crates/ml", default-features = false, features = ["financial", "cuda", "validation-mod"] }
data.workspace = true
common = { workspace = true, features = ["database"] }

2
services/trading_service/Cargo.toml
View File

@@ -84,7 +84,7 @@ risk.workspace = true
# 8 leaf-level ml-* sub-crates (ml-backtesting, ml-paper-trading, ml-stress-testing,
# ml-explainability, ml-universe, ml-regime-detection, ml-validation,
# ml-data-validation) that no source file in this service references.
ml = { path = "../../crates/ml", default-features = false, features = ["financial", "cuda"] }
ml = { path = "../../crates/ml", default-features = false, features = ["financial", "cuda", "validation-mod"] }
data.workspace = true
common = { workspace = true, features = ["database"] }
config = { workspace = true, features = ["postgres"] }