-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
scikit-learn 1.0+ compatible release on PyPI #1371
Comments
Hello @auvipy It's on our todo list once we get some other PR's that we are currently working on, if youd like to start, we would be very much appreciative. Doing so would likely require a decent amount of work, more than just updating the requirements file. It would involve going through the changelog and updating each component as neccessary and updating any test values that would change. At that point, we have external tests that validate that autosklearn does not degrade performance on automlbenchmark which we would perform externally from github. @mfeurer, do you ideas of what else would be required? Best, |
ok i am also up for coding changes as well |
Thanks @auvipy, I'll help guide you along the way and answer questions so feel free to open a PR and continue discussion there! Best, |
Hey @eddiebergman we could actually also try a more gradual path in which we first allow both scikit-learn version in parallel and update the individual components over time? |
@mfeurer, @eddiebergman is there any timeline for this issue? |
I'm willing to restart the effort |
Yes, it's not released yet, just saying that the refactored auto-sklearn will already use the latest scikit learn (once it's released) |
are they work in progress? shouldn't we consider any external contribution to the the project in the time being? |
We can, but we would hesitate actually encouraging it, as updating sklearn has already been done in the refactor and so has dropping Python 3.7. Apologies for having it private as there are reasons outside our control for doing so until it is released. I will update the main issue once there is an update. |
I will be happy to contribute
The text was updated successfully, but these errors were encountered: