scikit-learn_bench icon indicating copy to clipboard operation
scikit-learn_bench copied to clipboard

benchmarking the linear kernel in SVC

Open amueller opened this issue 6 years ago • 2 comments

Using kernel='linear' in SVC basically makes no sense. You should be using LinearSVC and probably set dual=False.

amueller avatar Jul 13 '19 20:07 amueller

DAAL will only work in the dual space for SVM, so daal4py's patches patch SVC instead of LinearSVC. We could possibly add an option to benchmark LinearSVC instead, or only choose SVC by default if DAAL is present. What do you think?

bibikar avatar Jan 31 '20 18:01 bibikar

The point is not what DAAL does but whether the benchmark is fair. If you want to compare solving a linear SVM between daal and scikit-learn, you should be using LinearSVC on the sklearn side.

amueller avatar Jan 31 '20 18:01 amueller