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
sklearn update fix #134
sklearn update fix #134
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ugh, does flake complain about nonused variables called _
this one was actually sensible. but there's something else going on on master. |
ah. scikit-learn recently upgraded. that seems to cause the error ... ... we might need to test against multiple versions :( |
@CorXccelerated just a heads up, this seems to affect the debug pipeline. |
so ... to me it seems like the testing for the debug pipeline is due for a bit of a revamp. the tests are super long. it does seem like some of the standard error messages out of scikit learn have changed. |
@MBrouns @CorXccelerated i don't suppose tomorrow y'all are going to be hacking on this :) ? |
No description provided.