Skip to content

Commit

Permalink
style: add comment about a fast_ica workaround
Browse files Browse the repository at this point in the history
  • Loading branch information
Lopa10ko committed May 29, 2024
1 parent 640a43c commit aa2fc5d
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,9 @@ def fit(self, input_data: InputData) -> PCA:

if self.number_of_features > 1:
self.check_and_correct_params(is_ts_data=input_data.data_type is DataTypesEnum.ts)
# TODO: remove a workaround by refactoring other operations in troubled pipelines (e.g. topo)
# workaround for NaN-containing arrays during pca fitting, especially for fast_ica
# fast_ica cannot fit with features represented by a rather sparse matrix
try:
self.pca.fit(input_data.features)
except Exception as e:
Expand Down

0 comments on commit aa2fc5d

Please sign in to comment.