From bfe5f72b0d0e9c59c3920164c965f2e5ad414caf Mon Sep 17 00:00:00 2001 From: Martin Vu <127065329+VuMartin@users.noreply.github.com> Date: Thu, 3 Sep 2026 01:51:32 +0000 Subject: [PATCH] fix(sklearn): rename SVR shrinking parameter (#7973) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ### What changes were proposed in this PR? Renamed the `probability` parameter to `shrinking` in `SklearnAdvancedSVRParameters`. The generated Python parameter was already correctly set to `shrinking`; only the displayed parameter name was incorrect. Before: Screenshot 2026-08-25 at 1 12 24 AM After: Screenshot 2026-08-25 at 1 35 53 AM **Note:** This renames the enum constant from `probability` to `shrinking`. Existing workflows saved with `probability` will need to be updated. ### Any related issues, documentation, discussions? Fixes #7949 ### How was this PR tested? Tested the Advanced SVM Regressor Trainer with `shrinking = true` and verified that the resulting parameters show `shrinking = True`. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: ChatGPT (5.5 GPT) (backported from commit ce121330fa0d781258b85e3de7f8512b56e47a21) --- .../SVRTrainer/SklearnAdvancedSVRParameters.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/workflow-operator/src/main/scala/org/apache/texera/amber/operator/machineLearning/sklearnAdvanced/SVRTrainer/SklearnAdvancedSVRParameters.java b/common/workflow-operator/src/main/scala/org/apache/texera/amber/operator/machineLearning/sklearnAdvanced/SVRTrainer/SklearnAdvancedSVRParameters.java index 898db274dc8..110b0f4a68d 100644 --- a/common/workflow-operator/src/main/scala/org/apache/texera/amber/operator/machineLearning/sklearnAdvanced/SVRTrainer/SklearnAdvancedSVRParameters.java +++ b/common/workflow-operator/src/main/scala/org/apache/texera/amber/operator/machineLearning/sklearnAdvanced/SVRTrainer/SklearnAdvancedSVRParameters.java @@ -28,7 +28,7 @@ public enum SklearnAdvancedSVRParameters implements ParamClass { degree("degree", "int"), coef0("coef0", "float"), tol("tol", "float"), - probability("shrinking", "(lambda value: value.lower() == \"true\")"), + shrinking("shrinking", "(lambda value: value.lower() == \"true\")"), verbose("verbose", "(lambda value: value.lower() == \"true\")"), epsilon("epsilon", "float"), cache_size("cache_size", "int"),