Evalml: 클래스 계층에서 "리프" 자식이 아닌 구성 요소 허용

옵션 :
1.) TargetImputer가 SimpleImputer를 사용하도록 합니다.
2.) TargetImputer 구성 요소를 정의하지 말고 대신 SimpleImputer를 대상에 적용하십시오.
3.) 정적 구성 요소 목록을 위해 _get_subclasses ()를 삭제합니다. (선호)
4.) _get_subclasses()를 더 똑똑하게 만듭니다.

다음 단계: 이 작업을 수행하는 방법을 결정하기 위해 토론 일정을 잡습니다.

내 의견은 우리가 코드베이스에서 기본이 아닌 클래스의 상속을 보다 일반적인 패턴으로 만들고 싶지 않다면 아무것도 변경해서는 안 된다는 것입니다. 우리는 현재 그렇게 하지 않기 때문에 그것을 허용하는 시스템을 구축하는 것은 나에게 낮은 우선순위로 보입니다.

이 문제는 목표 전가 자나 우리가 앞두고있는 단기 작업과 하위 클래스 "작동"을 식별하는 현재 시스템을 차단하지 않습니다. 우리가 정말로 3을 원한다면 그것은 또 다른 이야기입니다.

그리고 미래에 TargetImputer에서 SimpleImputer를 활용하고 싶다면 상속보다 합성을 사용할 수 있습니다. 이것이 옵션 1이 참조하는 것입니다.

여기서 @freddyaboulton에 동의합니다. 또한 정적 구성 요소 목록을 추적하는 것이 점점 더 많은 구성 요소를 추가함에 따라 약간 지루하고 실망 _get_subclasses() 를 정의했다는 점에 주목할 가치가 있습니다. 3 개를 사용하는 경우 _get_subclasses() 를 유지하는 것이 더 합리적 일 수 있지만 SimpleImputer / TargetImputer 케이스와 같은 "예외"목록도 유지하여 포함되도록하지만 Transformer 은 (는) 여전히 제외됩니다.

작업이 완료되면 작곡과 같은 기법에 대한 아이디어도 괜찮습니다.

@angela97lin 이걸 닫아도 될까요? TargetImputer / 기타 구성 요소를 개발하기 위해 _get_subclasses 를 변경할 필요가 없다는 데 동의합니다.