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

Add support for categorical features of any type #2242

Closed
wants to merge 1 commit into from
Closed

Add support for categorical features of any type #2242

wants to merge 1 commit into from

Conversation

eliseygusev
Copy link

Added auto-conversion of the user passed categorical columns to pandas.category. This solves this issue and another step towards this

@msftclas
Copy link

msftclas commented Jun 27, 2019

CLA assistant check
All CLA requirements met.

@guolinke guolinke requested a review from StrikerRUS August 24, 2019 23:30
@StrikerRUS
Copy link
Collaborator

@eliseygusev Hi! Thanks for your PR!

The title says

Add support for categorical features of any type

Unfortunately, it's not reachable without changing underlying JSON serializer, refer to #2134 (comment). Can you please describe the intention of this PR in more details?

Also, please rebase the branch and fix conflicts.

@jameslamb
Copy link
Collaborator

Hi @eliseygusev , I am going to close this PR due to inactivity. When you find time to come back to it, please rebase to update it to current master, re-open the PR, and respond to #2242 (comment). We'll be happy to review the changes at that time.

Thanks!

@jameslamb jameslamb closed this Jan 27, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants