alteryx / alteryx/evalml

Refactor LDA/PCA classes

Offen
#2,197 0 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @asniyaz Auf GitHub ansehen
refactor spike
Vorherrschende Sprache
Python
Sterne
850
Forks
96
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

LDA and PCA have a lot of common code. It seems these could be boiled down to a common base class that has `.fit()` `.fit_transform()` and `.transform()` defined with the individual classes just overriding `__init__()`. It seems that allowing `infer_feature_types()` to handle None is a prerequisite to this.

Is this worthwhile?

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.