Add DecisionTreeClassifier and DecisionTreeRegressor to AutoML#1255
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1255 +/- ##
=======================================
Coverage 99.93% 99.93%
=======================================
Files 208 208
Lines 13211 13211
=======================================
Hits 13203 13203
Misses 8 8
Continue to review full report at Codecov.
|
freddyaboulton
left a comment
There was a problem hiding this comment.
@angela97lin I think this looks great! I left a comment on the performance tests explaining why I think it's ok to add these estimators to search.
dsherry
left a comment
There was a problem hiding this comment.
@angela97lin awesome!!! Love the analysis here.
I'd like to understand a bit more about what those 4 outlier datasets were in the validation accuracy graph. That said, I'd be shocked if those had anything to do with decision trees -- I suspect a bug with looking glass, or possibly a bug with automl. So, no need to hold merge for that. But let's discuss.
Closes #1236
Quip doc here: https://alteryx.quip.com/u5KjAUW92aeS/Adding-DecisionTree-Estimators-in-EvalML