Update requirements to add sklearn minimum version

mlflow / recipes-examples

Example repo to kickstart integration with mlflow recipes.

Apache License 2.0

37 stars 49 forks source link

Update requirements to add sklearn minimum version #15

Closed sunishsheth2009 closed 2 years ago

sunishsheth2009 commented 2 years ago

Signed-off-by: Sunish Sheth sunishsheth2009@gmail.com

jinzhang21 commented 2 years ago

The integration tests are failing it appears.

jinzhang21 commented 2 years ago

Once we merge this should we revert #13? Would no longer be needed if we enforce >=1.1

No we cannot. This is a requirement for the example repo. Users can still use whatever they want, including old sklearn versions. We should support that.