[ML] Add check for loss_function when cloning a regression job (elast…

…ic#65481)
peteharverson · May 6, 2020 · 97aa353 · 97aa353
1 parent 582c986
commit 97aa353
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/...frame_analytics/pages/analytics_management/components/analytics_list/action_clone.test.ts b/...frame_analytics/pages/analytics_management/components/analytics_list/action_clone.test.ts
@@ -92,6 +92,7 @@ describe('Analytics job clone action', () => {
             training_percent: 20,
             randomize_seed: -2228827740028660200,
             num_top_feature_importance_values: 4,
+            loss_function: 'mse',
           },
         },
         analyzed_fields: {
@@ -192,6 +193,7 @@ describe('Analytics job clone action', () => {
             training_percent: 20,
             randomize_seed: -2228827740028660200,
             num_top_feature_importance_values: 4,
+            loss_function: 'mse',
           },
         },
         analyzed_fields: {

diff --git a/...ata_frame_analytics/pages/analytics_management/components/analytics_list/action_clone.tsx b/...ata_frame_analytics/pages/analytics_management/components/analytics_list/action_clone.tsx
@@ -179,6 +179,10 @@ const getAnalyticsJobMeta = (config: CloneDataFrameAnalyticsConfig): AnalyticsJo
               // By default it is randomly generated
               ignore: true,
             },
+            loss_function: {
+              optional: true,
+              defaultValue: 'mse',
+            },
           },
         }
       : {}),