janzd / CRNN

Convolutional recurrent neural network for scene text recognition or OCR in Keras
MIT License
122 stars 33 forks source link

Requirements #7

Open ErolCitak opened 4 years ago

ErolCitak commented 4 years ago

Hello,

I'm trying to evaluation but i'm getting some errors which is related to tensorflow, keras versions. Can you provide requirements with their versions if possible.

Thanks

andresmunozb commented 4 years ago

I have the same problem. Aiuda.

andresmunozb commented 4 years ago

I changed the imports for tensorflow.keras and i did another thing because it is not enough. You hace to change the name of one variable in this line https://github.com/kurapan/CRNN/blob/90df957f5fd8e19ec933f3e9c9d3c46741c89a4f/STN/spatial_transformer.py#L36 i just put this instead that: self.trainable_weights1 = self.locnet.trainable_weights and that's all!

andresmunozb commented 4 years ago

i did an api for with this code, idk how works the legal licenses with this but you can see it in here.

if you know if i have to put something in my github for the license of the code please tell me.

janzd commented 4 years ago

if you know if i have to put something in my github for the license of the code please tell me.

I put MIT license to this repo so that it can be pretty much freely reused. You could do the same if you want. It should grant free reuse of the code without any liability.

I changed the imports for tensorflow.keras and i did another thing because it is not enough. You hace to change the name of one variable in this line

Do you use Keras API included in TensorFlow? I wrote this code with a standalone Keras installation and TF backend so making it work for tensorflow.keras certainly requires some changes.