BUG rm RidgeClassifier from 20newsgroups

larsmans · GaelVaroquaux · commit 02e6fd07fe67 · 2012-10-06T15:52:28.000+02:00
Current implementation has quadratic space complexity in terms of X. Should use scipy.sparse.linalg.lsqr for linear space version (@mblondel).
diff --git a/examples/document_classification_20newsgroups.py b/examples/document_classification_20newsgroups.py
@@ -32,7 +32,6 @@
 from sklearn.datasets import fetch_20newsgroups
 from sklearn.feature_extraction.text import TfidfVectorizer
 from sklearn.feature_selection import SelectKBest, chi2
-from sklearn.linear_model import RidgeClassifier
 from sklearn.svm import LinearSVC
 from sklearn.linear_model import SGDClassifier
 from sklearn.linear_model import Perceptron
@@ -191,8 +190,7 @@ def benchmark(clf):
 
 
 results = []
-for clf, name in ((RidgeClassifier(tol=1e-1), "Ridge Classifier"),
-                  (Perceptron(n_iter=50), "Perceptron"),
+for clf, name in ((Perceptron(n_iter=50), "Perceptron"),
                   (KNeighborsClassifier(n_neighbors=10), "kNN")):
     print 80 * '='
     print name
