muffnn icon indicating copy to clipboard operation
muffnn copied to clipboard

move MLPClassifier to multilabel, multiclass

Open beckermr opened this issue 7 years ago • 5 comments

beckermr avatar May 18 '17 12:05 beckermr

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.

stephen-hoover avatar Feb 14 '18 16:02 stephen-hoover

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?

mheilman avatar Feb 14 '18 17:02 mheilman

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

beckermr avatar Feb 14 '18 17:02 beckermr

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

beckermr avatar Feb 14 '18 17:02 beckermr

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.

stephen-hoover avatar Feb 14 '18 18:02 stephen-hoover