You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
I couldn't find the original reference of participation ratio, maybe you know it ? It seems usual practice to refer to PR of eigenvalues of the covariance matrix. But I can add an option to return eigenvalues without centering if you need it for your application, thanks for the suggestion!
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 bysquare(sum(eigenvalues)) / sum(square(eigenvalues))
ofX^T X
, but the implementation usesPCA
.scikit-dimension/skdim/id/_PCA.py
Line 149 in 1ddee0f
This is incorrect because
PCA
first demeans the data. I don't think that's correct.The text was updated successfully, but these errors were encountered: