fix empty string in class_prior

Author: Marigold

python-sgd-regression/sgd_regression.py

@@ -174,24 +174,36 @@ def _parse_parameters(parameters):
         if name == 'class_prior':
             if value is not None:
                 value = str(value).strip()
-                if value != '':
+                if value == '':
+                    values = None
+                else:
                     try:
                         values = [float(v) for v in value.replace(' ', '').split(',')]
                     except ValueError:
                         raise errors.UserError('Wrong format {} for class_prior'.format(value))
                     if sum(values) != 1:
                         raise errors.UserError('Values in class_prior must sum to 1 ({} given)'.format(values))
-                    parameters[name] = values
+            else:
+                values = None
+
+            parameters[name] = values
 
         elif name == 'hidden_layer_sizes':
             if value is not None:
                 value = str(value).strip()
-                if value != '':
+                if value == '':
+                    # default value
+                    values = (100, )
+                else:
                     try:
                         values = [int(v) for v in value.replace(' ', '').split(',')]
                     except ValueError:
                         raise errors.UserError('Wrong format {} for hidden_layer_sizes'.format(value))
-                    parameters[name] = values
+            else:
+                # default
+                values = (100, )
+
+            parameters[name] = values
 
         else:
             # try converting it to float or integer if possible

python-sgd-regression/tests/docker-compose.yml

@@ -82,8 +82,9 @@ services:
       PARAM_covariables: "minimentalstate,opticchiasm,subjectageyears"
       PARAM_grouping: ""
       PARAM_meta: "{\"lefthippocampus\":{\"code\":\"lefthippocampus\",\"type\":\"real\",\"mean\":3.0,\"std\":0.35,\"minValue\":0.1,\"maxValue\":5.0},\"minimentalstate\":{\"code\":\"minimentalstate\",\"type\":\"real\",\"mean\":24.0,\"std\":5.0},\"opticchiasm\":{\"code\":\"opticchiasm\",\"type\":\"real\",\"mean\":0.08,\"std\":0.009},\"subjectage\":{\"code\":\"subjectage\",\"type\":\"real\",\"mean\":71.0,\"std\":8.0}, \"rs17125944_c\": {\"code\": \"rs17125944_c\",\"enumerations\": [{\"code\": 0,\"label\": 0},{\"code\": 1,\"label\": 1},{\"code\": 2,\"label\": 2}],\"sql_type\": \"int\",\"type\": \"polynominal\"}, \"adnicategory\": {\"code\": \"adnicategory\", \"enumerations\": [{\"code\": \"AD\", \"label\": \"Alzheimer's Disease\"}, {\"code\": \"MCI\", \"label\": \"Mild Cognitive Impairment\"}, {\"code\": \"CN\", \"label\": \"Cognitively Normal\"}], \"type\": \"polynominal\"}, \"alzheimerbroadcategory\": {\"code\": \"alzheimerbroadcategory\", \"enumerations\": [{\"code\": \"AD\", \"label\": \"Alzheimer's disease\"}, {\"code\": \"CN\", \"label\": \"Cognitively Normal\"}, {\"code\": \"Other\", \"label\": \"Other\"}], \"type\": \"polynominal\"}, \"subjectageyears\": {\"code\": \"subjectageyears\", \"label\": \"Age Years\", \"maxValue\": 130, \"minValue\": 0, \"type\": \"integer\"}}"
-      MODEL_PARAM_type: "linear_model"
-      MODEL_PARAM_alpha: 0.25
+      MODEL_PARAM_type: "naive_bayes"
+      MODEL_PARAM_class_prior: ""
+      MODEL_PARAM_alpha: 1
 
   sgd-regression-a:
     extends: sgd-regression-base