search

jamesmf / mnistCRNN

Simple TimeDistributed() wrapper Demo in Keras; sums images of MNIST digits
61 stars 24 forks source link

Keras 0.3.2 compatibility and better RMSE #3

Closed GregorySenay closed 8 years ago

GregorySenay commented 8 years ago

Hi @jamesmf, I make quicks modifications to be fully compatible with Keras 0.3.2 without any additional or extra layers. You can verify it but the model provides better RMSE. If you are okay, it can be a very good example to be integrated in Keras? What do you think? Thx,

jamesmf commented 8 years ago

This looks great, thanks for the update. I'd be happy to have it be an example if that's useful.

GregorySenay commented 8 years ago

Great @jamesmf ! Did you also verify the RMSE value with this model? Normally it's better, if you do it, It will make a second check for this model. And can you update the README with the new values if you do it?

jamesmf commented 8 years ago

Updated README, the RMSE improved slightly.

GregorySenay commented 8 years ago

Great ! Thanks for confirm it