Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect number of classes #39

Open
maykulkarni opened this issue Jan 31, 2020 · 4 comments
Open

Incorrect number of classes #39

maykulkarni opened this issue Jan 31, 2020 · 4 comments

Comments

@maykulkarni
Copy link

maykulkarni commented Jan 31, 2020

Hey thanks for sharing the code, but I found a possible issue while training the network. While editing the number of characters in the alphabet.py file, I followed the guide and replaced the Chinese characters by English one and my network trained fine. But while reading the code and debugging I found that the nClass output dimension of the CRNN was 72 while the number of unique characters in the alphabet.py class was only 36. I eventually realized that that code is splitting the characters wrongly and considering \n newline as a character as well that's why the output dimension was [26x1x72] instead of [26x1x37] this can cause an issue in training. I can raise a PR fixing this if you want. Thanks.

@Holmeyoung
Copy link
Owner

Of course you can. Thank you~

@livekeys
Copy link

Hello, how can I solve this problem!

@maykulkarni
Copy link
Author

maykulkarni commented Mar 24, 2020

@livekeys just edit the alphabet.py to have all the characters in the same line this way \n will not be treated as a new character.

@livekeys
Copy link

Thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants