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

We must choose the network when watching it play using the ale_run_watch.py script #61

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

yzsatgithub
Copy link

Added a sys.argv to ale_run_watch.py to choose the network used to play, because this scripts runed the trained network only on the Nature network(run_nature.py). If I trained the network with the NIPS one, we won't see the agent play the game properly when running the ale_run_watch.py script.

So I add a sys.argv(nips/nature) to ale_run_watch.py and add a little change in the README.md.

Please review it and see if it works. And please tell me if my English isn't good enough to make sense :)

… use params trained by nips to run nature network.
… use params trained by nips to run nature network.
@gy2256
Copy link

gy2256 commented Mar 9, 2017

yeah, This is actually quiet important

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.

2 participants