Skip to content

Old test for pull request #21753

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

Closed
wants to merge 2 commits into from
Closed

Old test for pull request #21753

wants to merge 2 commits into from

Conversation

jfreneau
Copy link

@jfreneau jfreneau commented Nov 23, 2021

Reference Issues/PRs

Fixes type problems for get_feature_names in OneHotEncoder

Hello, I have to work on this issue for a school project.
I propose to cast integer-columns to string-columns & put a warning to inform the user that it is possible that others errors will happen later.

I don't think we should force the user to put strings because some of the cases stated above are common (like csv files without headers). We can't change all the consistency problems of Pandas.

- Check that the type of the elements of input_features is string. If not raise a warning explaining that others problem could occur. 
- Cast input_features elements to string
@jfreneau jfreneau closed this Nov 23, 2021
@jfreneau jfreneau changed the title Fixes #16593 Old test for pull request Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant