alteryx / alteryx/evalml

Update label encoder in pipeline implementation to look for LabelEncoder class instead of looking for "Label Encoder"

オープン
#2,906 コメント 1 件 リアクション 1 件 担当者 1 名 @asniyaz が担当を希望しています GitHub で見る
enhancement needs design tech debt
主要言語
Python
スター
850
フォーク
96
PR マージ指標
30日以内にマージされた PR はありません

説明

https://github.com/alteryx/evalml/pull/2821/ introduces the label encoder as a component in the pipeline, rather than an attribute. However, we still set a pipeline's `_encoder` attribute to a label encoder in the pipeline, if it's available.

As documented in [this comment](https://github.com/alteryx/evalml/pull/2821/files#r723692569), there is an assumption that the Label Encoder component has a name "Label Encoder". However, we should be more flexible and accept the label encoder even if its name was "my cool label encoder".

One suggestion is to find the component by checking for the component class instead. This work would be made much easier by https://github.com/alteryx/evalml/issues/2878.

One edge case that may be important to think about the case where there may be multiple label encoders. What do we do then?

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。