Accelerate LU
when input has one chunk
#905
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What do these changes do?
As described in #904 , when input has one chunk, execution of
LU
is super slow. This PR accelerated the process.Because
LU
is used in randomized_svd, and the latter one is widely used inPCA
andTruncatedSVD
, I test the unittests intest_pca
, the running time reduced from 5min 14s to 1min 35s on my laptop. I think this PR can accelerate the CI as well.Related issue number
Fixes #904 .