socialfoundations / folktables

Datasets derived from US census data
MIT License
240 stars 19 forks source link

Refer to scikit-learn with full PyPI package name (fix #32) #33

Closed jansim closed 1 year ago

jansim commented 1 year ago

This should resolve #32 (I don't know the package source that well, but it seemed to be the only occurence of sklearn I could find).

From what it looked like, I think scikit-learn could also be moved to be a development dependency if it is only needed to run tests? (in extras_require in setup.py)? 🤔

mrtzh commented 1 year ago

Thanks!