generated from scikit-learn-contrib/project-template
-
Notifications
You must be signed in to change notification settings - Fork 22
Open
Description
I just noticed what I think might be a problem with your implementation of Participation Ratio!
Suppose one is given a matrix X with shape (num samples, num features). The PR is given by square(sum(eigenvalues)) / sum(square(eigenvalues)) of X^T X, but the implementation uses PCA.
scikit-dimension/skdim/id/_PCA.py
Line 149 in 1ddee0f
| pca = PCA().fit(X) |
This is incorrect because PCA first demeans the data. I don't think that's correct.
Metadata
Metadata
Assignees
Labels
No labels