simple_dqn icon indicating copy to clipboard operation
simple_dqn copied to clipboard

./play.sh snapshots/space_invaders_200.pkl error out

Open mw66 opened this issue 8 years ago • 2 comments

./play.sh got the rom name wrong, it invokes:

python src/main.py --play_games 1 --display_screen true --load_weights snapshots/space_invaders_200.pkl roms/space.bin

while it should be space_invaders.bin

mw66 avatar Jun 26 '16 18:06 mw66

Yes, this problem was already raised in pull request #6. But I couldn't accept the proposed fix, because it broke the (optional) experiment label feature. I guess different separator should be used for experiment label, for example -. I will keep it open until I have time to fix it, you're welcome to submit a pull request.

tambetm avatar Jun 27 '16 08:06 tambetm

train.sh works :-)

./train.sh roms/space_invaders.bin

python src/main.py --save_weights_prefix snapshots/space_invaders --csv_file results/space_invaders.csv roms/space_invaders.bin

game=${full%.*}

instead of

game=${full%%_*}

mw66 avatar Jun 28 '16 04:06 mw66