BUG use ValueError in BaseLibSVM.coef_

larsmans · larsmans · commit 722fef6b4236 · 2012-01-08T20:00:37.000+01:00
NotImplementedError is for abstract methods.
diff --git a/sklearn/svm/base.py b/sklearn/svm/base.py
@@ -113,8 +113,8 @@ def predict_log_proba(self, T):
     @property
     def coef_(self):
         if self.kernel != 'linear':
-            raise NotImplementedError('coef_ is only available when using a '
-                                      'linear kernel')
+            raise ValueError('coef_ is only available when using a '
+                             'linear kernel')
         coef = safe_sparse_dot(self.dual_coef_, self.support_vectors_)
         # coef_ being a read-only property it's better to mark the value as
         # immutable to avoid hiding potential bugs for the unsuspecting user
