Evalml: السماح للمكونات التي ليست "طرفية" فرعية في التسلسل الهرمي للفئة

خيارات:
1.) اجعل TargetImputer تستخدم SimpleImputer.
2.) لا تحدد مكون TargetImputer ، ولكن بدلاً من ذلك قم بتطبيق SimpleImputer على الهدف.
3.) احذف _get_subclasses () لصالح قائمة المكونات الثابتة. (يفضل)
4.) اجعل _get_subclasses () أكثر ذكاءً.

الخطوة التالية: حدد موعدًا لمناقشة لتحديد كيفية القيام بذلك.

رأيي أنه لا ينبغي لنا تغيير أي شيء ما لم نرغب في جعل وراثة الفئات غير الأساسية نمطًا أكثر شيوعًا في قاعدة الرموز الخاصة بنا. نحن لا نقوم بذلك حاليًا ، لذا فإن بناء نظام للسماح بذلك يبدو أولوية منخفضة بالنسبة لي.

هذه المشكلة لا تمنع الحاجز المستهدف أو أي عمل قصير المدى أمامنا والنظام الحالي لتحديد الفئات الفرعية "يعمل". إذا كنا نريد حقًا 3 ، فهذه قصة أخرى وسأكون محبطًا لسماع الحجج المؤيدة لها ولكن لا أعتقد أن هناك حاجة للقيام بذلك.

وإذا أردنا الاستفادة من SimpleImputer في TargetImputer في المستقبل ، فيمكننا استخدام التركيب على الميراث ، والذي أعتقد أنه ما يشير إليه الخيار 1.

تم الاتفاق مع freddyaboulton هنا ، وتجدر الإشارة أيضًا إلى أننا حددنا _get_subclasses() في المقام الأول لأن تتبع قائمة ثابتة من المكونات كان مملاً / محبطًا بعض الشيء لأننا أضفنا المزيد والمزيد من المكونات. إذا ذهبنا إلى 3 ، فقد يكون من المنطقي الاحتفاظ بـ _get_subclasses() ولكن أيضًا الاحتفاظ بقائمة من "الاستثناءات" مثل علبة SimpleImputer / TargetImputer ، بحيث يتم تضمينها ، ولكن هناك مكونات أخرى مثل Transformer لا تزال مستبعدة.

حسنًا أيضًا مع فكرة بعض تقنيات التكوين إذا كان ذلك ينجز المهمة.

@ angela97lin هل تعتقد أنه يمكننا إغلاق هذا؟ أعتقد أننا نتفق على أننا لسنا بحاجة إلى إجراء تغييرات على _get_subclasses لتمكين تطوير TargetImputer / المكونات الأخرى.