keplr-io / hera

Train/evaluate a Keras model, get metrics streamed to a dashboard in your browser.
MIT License
487 stars 47 forks source link

Update callback.py #13

Closed igorbb closed 7 years ago

igorbb commented 7 years ago

Changed line of -- from keras import backend as K -- to avoid indentation error.

igorbb commented 7 years ago

Pull request is solution to issue : #12

jakebian commented 7 years ago

Did you verify that this fixes your issue in your environment? It's a bit of a bizarre bug and to me signals something's wrong with your setup.

It looks like we can remove that import altogether as we're not actually using it - we should do that instead.

igorbb commented 7 years ago

Did you verify that this fixes your issue in your environment? It's a bit of a bizarre bug and to me signals something's wrong with your setup.

Yes . With This modification it works well.

I also tough the error was weird. But its a clean docker setup for running Keras.

It looks like we can remove that import altogether as we're not actually using it - we should do that instead.

Indeed. I did not read the code with a 100 of attention, and never realized that the backend is not being used.