Skip to content
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

Add redis to requirements in README #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add redis to requirements in README #6

wants to merge 1 commit into from

Conversation

MartinThoma
Copy link

Redis is required by tf_rl/controller/human_controller.py

Redis is required by tf_rl/controller/human_controller.py
@imcomking
Copy link

sudo apt-get install redis-server
sudo service redis-server start
(or sudo service redis-server restart)

@imcomking
Copy link

if you want simulate HumanController, you should add some codes in human_controller.py first.

import sys, os
sys.path.append(os.path.abspath('../..'))

python human_controller.py
and push some keyboard actions (i.e w,a,s,d)
then run the ipython notebook "karpathy_game.ipynb"

@lsqshr
Copy link
Contributor

lsqshr commented Jun 7, 2016

+1

Also due to the absolute importing, it now only works if you copy the human_controller.py to the project root

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants