Ji-Zhang / datacleanbot

MIT License
8 stars 1 forks source link

Bayesian requires pyro which can't be installed #1

Closed amueller closed 5 years ago

amueller commented 5 years ago

I'm trying to import dataclean, but it looks like the bayesian subpackage requires pyro. pyro doesn't seem to be in requirements.txt and pip install pyro fails (seems like it might be a python2 package?)

amueller commented 5 years ago

looks like the required package might be pyro-ppl?

amueller commented 5 years ago

rpy2 is also a dependency that's not documented

Ji-Zhang commented 5 years ago

looks like the required package might be pyro-ppl?

Yes, just updated that. Thanks!

Ji-Zhang commented 5 years ago

rpy2 is also a dependency that's not documented

Thanks. Updated already. And now the package has been published such that it can be installed via pip install datacleanbot.

amueller commented 5 years ago

that's great, thanks! The dependencies are still a bit heavy but I guess that's hard to change (theano, torch and tensorflow are all required).