Evalml: Komponenten zulassen, die keine "Blatt"-Kinder in der Klassenhierarchie sind

Optionen:
1.) Lassen Sie den TargetImputer den SimpleImputer verwenden.
2.) Definieren Sie keine TargetImputer-Komponente, sondern wenden Sie stattdessen den SimpleImputer auf das Ziel an.
3.) Löschen Sie _get_subclasses() zugunsten einer statischen Liste von Komponenten. (bevorzugt)
4.) Machen Sie _get_subclasses() intelligenter.

Nächster Schritt: Planen Sie eine Diskussion, um zu entscheiden, wie dies zu tun ist.

Meiner Meinung nach sollten wir nichts ändern, es sei denn, wir möchten die Vererbung von Nicht-Basisklassen zu einem häufigeren Muster in unserer Codebasis machen. Das tun wir derzeit nicht, daher scheint mir der Aufbau eines Systems, das dies ermöglicht, nachrangig zu sein.

Dieses Problem blockiert nicht den Ziel-Imputer oder irgendeine kurzfristige Arbeit, die wir vor uns haben, und das aktuelle System zum Identifizieren von Unterklassen "arbeitet". Wenn wir wirklich 3 wollen, dann ist das eine andere Geschichte, hehe, und ich würde gerne Argumente dafür hören, aber ich denke nicht, dass es notwendig ist, es zu tun.

Und wenn wir in Zukunft den SimpleImputer im TargetImputer nutzen möchten, können wir die Komposition über die Vererbung verwenden, was meiner Meinung nach Option 1 betrifft.

Ich stimme @freddyaboulton hier zu, auch erwähnenswert, dass wir zuerst _get_subclasses() definiert haben, weil es etwas mühsam / frustrierend war, den Überblick über eine statische Liste von Komponenten zu behalten, da wir immer mehr Komponenten hinzugefügt haben. Wenn wir uns für 3 entscheiden, ist es möglicherweise sinnvoller, _get_subclasses() beizubehalten, aber auch eine Liste von "Ausnahmen" wie den SimpleImputer / TargetImputer-Fall zu führen, damit sie enthalten sind, aber andere Komponenten wie die Transformer sind noch ausgeschlossen.

Auch mit der Idee einer kompositorischen Technik einverstanden, wenn dies der Fall ist.

@angela97lin Glaubst du, wir können das schließen? Ich denke, wir sind uns einig, dass wir keine Änderungen an _get_subclasses vornehmen müssen, um die Entwicklung von TargetImputer/anderen Komponenten zu ermöglichen.