Corrections made to the config.yaml file and added Scikit test in benchmark_decision_tree.py.

mlpack / benchmarks

Machine Learning Benchmark Scripts

101 stars 49 forks source link

Corrections made to the config.yaml file and added Scikit test in benchmark_decision_tree.py. #56

Closed Iron-Stark closed 7 years ago

Iron-Stark commented 7 years ago

@zoq @rcurtin

The config.yaml file contains the decision tree shogun block under scikit library block.There was also an error in the scikit dtc.py implementation that also has been corrected here.Please Review.

mlpack-jenkins commented 7 years ago

Can one of the admins verify this patch?

rcurtin commented 7 years ago

Looks good to me, pretty simple changes; thanks for the fix.