-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
Fixed bug handling multi-class classification #19427
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
base: main
Are you sure you want to change the base?
Fixed bug handling multi-class classification #19427
Conversation
We would need to adjust the OVO related tests to also expect this shift by Also as discussed in #19270 (comment) we probably need to go through a deprecation cycle. |
#DataUmbrella |
@icky254 How is this PR going? Please let us know if we can answer any questions. cc: @reshamas |
I'm having a bit of a challenge. I seem to have lost my PR from my local repo. How can I get this PR to my repo? |
Hi @icky254,
assuming you have configured your remote as
will create a new branch named |
It's good now. Thanks. |
@isaack-mungui |
@reshamas I am working on it. I'll make a commit soon. |
I'm trying to run pytest, but getting the following error: I followed the steps in Contents of sklearn/__check_build: It seems that scikit-learn has not been built correctly. If you have installed scikit-learn from source, please do not forget It appears that you are importing a local scikit-learn source tree. For Where am I going wrong with this? |
Hi @isaack-mungui I have been there without knowing how to solve :(.
all files related to the build should disappear, but you can check that all |
@cmarmo I managed to fix it. I ran |
Reference Issues/PRs
#19270
What does this implement/fix? Explain your changes.
Fixes a bug in the function handling multi-class classification by reducing the fixed value of the ovr_decision_function, which is too large.