-
Notifications
You must be signed in to change notification settings - Fork 17
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
enforcing most recent build of scikit-learn #59
Conversation
We should also update the version in Why is that that the documentation needs a separate requirements.txt? It would be nice if it could use the same dependency already declared in |
d57c1d9
to
6aef8c2
Compare
@Luthaf updated the docs/requirements. Any thoughts on the failing tests? |
Looks like the project on pypi is not called sklearn but scikit-learn: https://pypi.org/project/sklearn/ points to https://pypi.org/project/scikit-learn/ |
6aef8c2
to
d3b8525
Compare
setup.cfg
Outdated
@@ -16,7 +16,7 @@ include_package_data = True | |||
zip_safe = True | |||
packages = find: | |||
install_requires = numpy | |||
sklearn | |||
sklearn>="0.24.0" |
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.
We should also change this one =)
sklearn added an arpack initializer in 0.24.0 that was previously unavailable.
d3b8525
to
599eb8d
Compare
It looks like we lost code coverage status though =/ |
What do you mean? |
sklearn added an arpack initializer in 0.24.0 that was previously unavailable.