-
Notifications
You must be signed in to change notification settings - Fork 11
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
Missing dependencies #109
Comments
@sonyahanson @jchodera Do you have any reservations about this change? |
These packages are not all We do have these listed in the conda package requirements, which is why we recommend people install this tool via conda. We haven't cut a release in quite some time, however. |
@jchodera Thanks for explaining why those are commented out. Do your recommend installing through conda and uninstalling to just get dependencies, and then installing assaytools from repository via When I tried conda installing assaytools with conda, it gives an error about Python 3.5 incompatibility.
I think we must update the conda release, so that it at least works with Python 3.5. |
Yes!
It looks like we haven't cut a new release since Python 3.5 was added. Let's create a new assaytools release. |
While installing assaytools in a Python 3.5 environment I noticed some dependencies are missing. I had to install
lxml
,seaborn
andpymc
separately after runningpython setup.py install
.I noticed that those missing dependencies are actually listed in
setup.py
, but commented out: https://github.com/choderalab/assaytools/blob/master/setup.py#L87I want to activate those lines so that these dependencies are also installed at setup.
The text was updated successfully, but these errors were encountered: