-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
No attribute classes_ during multi-class scoring #26336
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
Comments
Thanks for reporting. Could you provide what is the class of |
I had a look at the I think that it misses a method to wire the @property
def classes_(self):
return self.model.classes_ |
For completeness, the current implementation of scikit-learn/sklearn/utils/estimator_checks.py Line 2225 in 8521819
|
That does indeed fix the issue, I'll add that! Not sure if there is anything to fix at the |
... but at our end do you suggest we use this |
Yep
It makes sense to try it. If the tests are passing then, it is quite safe to use will any component of scikit-learn. Note that we always have a mechanism to skip some of the common tests if it fails for a good reason. |
Thanks for the quick response, and sorry that we should have caught that it was our faulty implementation in the end! |
Happy to be helpful ;) |
Describe the bug
Regression we hit in MNE-Python's
pip-pre
run and bisected locally:https://github.com/mne-tools/mne-python/actions/runs/4894775241/jobs/8739516519#step:17:4138
Local bisect suggests the culprit is #26037 by @glemaitre
It's entirely possible we're doing something wrong at the MNE-Python end but this code has worked for years 🤷
cc @agramfort who might also know the problem quickly
Steps/Code to Reproduce
Not really minimal, but after cloning MNE-Python you can do:
And it will fail.
Can try to whittle it down more if it would help...
Expected Results
No error
Actual Results
Versions
The text was updated successfully, but these errors were encountered: