Update requirements.txt for tensorflow-gpu

allenai / deep_qa

A deep NLP library, based on Keras / tf, focused on question answering (but useful for other NLP too)

Apache License 2.0

404 stars 133 forks source link

Update requirements.txt for tensorflow-gpu #241

Closed nelson-liu closed 7 years ago

nelson-liu commented 7 years ago

tensorflow-gpu seems to have the ability to run CPU code...so I think it is a reasonable default.

The main reason i made this change is because setting up the solver to run on docker installs from requirements.txt, and we probably want the GPU version on there.

nelson-liu commented 7 years ago

looks like it's mostly OK...just complaining that there's no CUDA. i'll force CPU in the travis config?

nelson-liu commented 7 years ago

nevermind, looks like it needs cuda even if it runs in CPU-only mode :/ yet another step to add to the already lengthy process of running the python server.