Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate OrdinalEncoder Component into AutoML #3744

Closed
tamargrey opened this issue Oct 6, 2022 · 1 comment · Fixed by #3765
Closed

Integrate OrdinalEncoder Component into AutoML #3744

tamargrey opened this issue Oct 6, 2022 · 1 comment · Fixed by #3765
Assignees

Comments

@tamargrey
Copy link
Contributor

Once #1389 is finished, the OrdinalEncoder will exist but not actually be used within EvalML. We should integrate it.

This will require a change to the OneHotEncoder, since when the ordinal encoder is used, the one-hot encoder should not encode ordinal columns.

@tamargrey
Copy link
Contributor Author

a thought I didn't want to forget: we use sklearn's OrdinalEncoder in the time series featurizer - maybe it'd be worth using ours instead, but then we'd need to change the ordinal encoder to use non ordinal categorical columns, so that's probably not what we want to do

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant