Merge pull request #250 from openml/develop

Merge dev into fix248

Author: janvanrijn

tests/test_runs/test_run_functions.py

@@ -9,7 +9,7 @@
 from openml.testing import TestBase
 from openml.runs.functions import _run_task_get_arffcontent
 
-from sklearn.tree import DecisionTreeClassifier
+from sklearn.tree import DecisionTreeClassifier, ExtraTreeClassifier
 from sklearn.preprocessing.imputation import Imputer
 from sklearn.dummy import DummyClassifier
 from sklearn.preprocessing import StandardScaler
@@ -166,6 +166,18 @@ def test_run_optimize_bagging_diabetes(self):
 
         run = self._perform_run(task_id, num_test_instances, grid_search)
         self.assertEqual(len(run.trace_content), num_iterations * num_folds)
+    
+    def test_run_with_classifiers_in_param_grid(self):
+        task = openml.tasks.get_task(115)
+
+        param_grid = {
+            "base_estimator": [DecisionTreeClassifier(), ExtraTreeClassifier()]
+        }
+
+        clf = GridSearchCV(BaggingClassifier(), param_grid=param_grid)
+        self.assertRaises(TypeError, openml.runs.run_task,
+                          task=task, model=clf, avoid_duplicate_runs=False)
+
 
         res = self._check_serialized_optimized_run(run.run_id)
         self.assertTrue(res)
@@ -373,3 +385,4 @@ def test_run_on_dataset_with_missing_labels(self):
         for row in data_content:
             # repeat, fold, row_id, 6 confidences, prediction and correct label
             self.assertEqual(len(row), 11)
+