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

move MLPClassifier to multilabel, multiclass #47

Open
beckermr opened this issue May 18, 2017 · 5 comments
Open

move MLPClassifier to multilabel, multiclass #47

beckermr opened this issue May 18, 2017 · 5 comments

Comments

@beckermr
Copy link
Contributor

No description provided.

@stephen-hoover
Copy link
Contributor

I've discovered that the MLPClassifier does not encode string targets when y is 2D. Is that part of this issue, or should I file a new issue? It's not clear what this issue covers.

@mheilman
Copy link
Contributor

That sounds like a separate issue.

@beckermr, could you clarify what this issue means? I believe it means adding support for multilabel-multiclass modeling. That right?

@beckermr
Copy link
Contributor Author

This was me thinking of using the list of lists format from sklearn. @stephen-hoover's question is a different issue.

@beckermr
Copy link
Contributor Author

@stephen-hoover Is that format supposed to be supported by sklearn API? I am reading the docs and it is very unclear.

@stephen-hoover
Copy link
Contributor

The reason to handle encoding for multilabel inputs is so that we can do multilabel-multiclass, so this issue sounds like it's still a blocker.

@beckermr , I agree it's unclear. Experimentally, the tree ensemble classifiers do correctly handle string input for multilabel-multiclass problems. There's a bug in the input checking which causes an error for target arrays with mixed string and numeric columns, and the predict method errors if you used string labels in the training. However fit and predict_proba work. I think we should support it in muffnn; it seems like a common use case to me.

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

No branches or pull requests

3 participants