From 669c3b93d4e5fa73eeb85fd1c538d9bd27d236e0 Mon Sep 17 00:00:00 2001 From: jgrusewski Date: Mon, 2 Mar 2026 14:17:54 +0100 Subject: [PATCH] fix(trading_service): update feature importance stub to unimplemented Changed from Status::unavailable to Status::unimplemented with path forward (ml_training_service forwarding). Fixed supports_feature_importance from true to false. Co-Authored-By: Claude Opus 4.6 --- services/trading_service/src/services/enhanced_ml.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/services/trading_service/src/services/enhanced_ml.rs b/services/trading_service/src/services/enhanced_ml.rs index 3cfe22dfa..f5bb9add7 100644 --- a/services/trading_service/src/services/enhanced_ml.rs +++ b/services/trading_service/src/services/enhanced_ml.rs @@ -874,7 +874,7 @@ impl MlService for EnhancedMLServiceImpl { capabilities: Some(ModelCapabilities { supports_streaming: true, supports_retraining: true, - supports_feature_importance: true, + supports_feature_importance: false, supports_confidence_intervals: false, supported_asset_classes: vec!["FX".to_string()], }), @@ -1167,8 +1167,8 @@ impl MlService for EnhancedMLServiceImpl { ) -> Result, Status> { let req = request.into_inner(); - Err(Status::unavailable(format!( - "Feature importance for '{}' requires SHAP or gradient-based computation — not yet implemented", + Err(Status::unimplemented(format!( + "Feature importance for '{}' requires forwarding to ml_training_service (integrated gradients available in ml::explainability)", req.model_name ))) }