Skip to content

Commit

Permalink
DOC remove confusing comment from TruncatedSVD
Browse files Browse the repository at this point in the history
The comment was supposed to suggest that there might be a faster
way to do fit_transform than the matrix-vector multiplication
that it currently performs, but the suggested alternative actually
has higher complexity.
  • Loading branch information
larsmans committed Nov 11, 2013
1 parent 215ca71 commit 317e97a
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion sklearn/decomposition/truncated_svd.py
Expand Up @@ -120,7 +120,6 @@ def fit_transform(self, X, y=None):
U, Sigma, VT = self._fit(X)
Sigma = np.diag(Sigma)

# or (X * VT.T).T, whichever takes fewer operations...
return np.dot(U, Sigma.T)

def _fit(self, X):
Expand Down

0 comments on commit 317e97a

Please sign in to comment.