greenelab / snorkeling

Extracting biomedical relationships from literature with Snorkel 🏊
Other
59 stars 17 forks source link

Upgrade environment to use snorkel v0.5 #6

Closed danich1 closed 7 years ago

danich1 commented 7 years ago

Closes https://github.com/greenelab/snorkeling/issues/5

dhimmel commented 7 years ago

Here's a screenshot of how snorkel's python-package-requirement.txt changes with the submodule update.

python-package-requirement.txt

As far as I can tell, the only change is that theano has been replaced by tensorflow. @ajratner will all uses need tensorflow? @danich1 so I think we should remove theano (check nothing breaks) and potentially add tensorflow, although we can hold off installing that until we need it, if you would like. Let's wait to hear from @ajratner.

ajratner commented 7 years ago

@dhimmel Yes we moved all the default end extraction models to TF in this release, so this would be the right way to go!

dhimmel commented 7 years ago

@danich1 let's see if tensorflow installs with pip. Otherwise, we can use conda and add the conda-forge channel to our environment.yml.

danich1 commented 7 years ago

Added the tensorflow install line. Pip installs it like a charm!!!