search

mwhittaker / deeprl_project

Deep RL Final Project
1 stars 1 forks source link

prevented tensorflow override by baselines #28

Open vlad17 opened 6 years ago

vlad17 commented 6 years ago

installing baselines kills a GPU tensorflow install. this commit fixes that dependency override.

Be careful: if you installed baselines before it may be using CPU only because it killed TF earlier.

the fix is to uninstall tf, tf-gpu, and then install baselines (if you haven't already), install tf-gpu.

otherwise, after this fix, following README Instructions should leave your system in the correct state.

SamvitJ commented 6 years ago

Will take a look. What do you mean by "kills a GPU tensorflow install"? Is a necessary package removed?

vlad17 commented 6 years ago

Yes, if you install as the instructions specified before, a baselines cpu-only tensorflow would clobber exising tensorflow installs.

vlad17 commented 6 years ago

ok; there's another problem, b/c the current README instr don't specify the openai/baselines commit to check out. adding a fix