Document h5py dependency and remove unused imports

yanpanlau / Keras-FlappyBird

Using Keras and Deep Q-Network to Play FlappyBird

435 stars 194 forks source link

Document h5py dependency and remove unused imports #12

Open KernelDeimos opened 7 years ago

KernelDeimos commented 7 years ago

This commit fixes a couple issues I encountered before successfully running qlearn.py.

Using Keras 2.0.2 I received an error message stating that keras.initializations module didn't exist. Looking through the source I found that the module wasn't used anywhere, so I commented it out and it worked.

Franck-Dernoncourt commented 7 years ago

+1 I was going to send the same pull request.

As a side note,in Keras 2.0, initializations was renamed as initializers. You should therefore instead write

from keras import initializers

which explains why you have the error message stating that keras.initializations module didn't exist. But as you noticed it is unused here anyway.