-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
[WIP] Added PredictionTransformer and ThresholdClassifier #6663
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
Conversation
Together these two classes can be used in a pipeline to change the classification threshold from the default of 0.5 to any value
Is there a case were you want one and not the other? |
Stacking is the one use case I had in mind. Train several classifiers, take each of their outputs and feed them into another model to "combine" them. I know this from HEP work (eg http://arxiv.org/abs/0903.0850 (p6 of the PDF), I'm sure there are more these guys were just the first I could find the paper for). Might be something people do in kaggle competitions?? Not sure it is "worth it" when taking into account your comment on public interfaces and removing them. Having two private classes with a third, public one, that does it all in one step sounds like a good move. |
Is there a way to mark my classifier as "two class only" for the common checks? Alternatively does someone have a good idea what Related to the stacking use case: #6674 |
@amueller any idea how to mark A good idea how to support multi class for |
Err also is |
we need this for two-class classifiers: #6599. For now, add it manually to one of the skip lists. Maybe just skip the multi-class tests? |
Aha! Yes, because I didn't read the docs for |
well, I knew it but still needed to reread the previous discussion to see that that was the case lol. |
Together these two classes can be used in a pipeline
to change the classification threshold from the default
of 0.5 to any value
This is a start on #4813
Is
pipeline.py
the right home for this?edit: from comment below:
Yes it is, should ditch
PredictionTransformer
from this PR./cc @joshlk