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

Vertical recognition #382

Closed
miliadis opened this issue Feb 25, 2021 · 2 comments
Closed

Vertical recognition #382

miliadis opened this issue Feb 25, 2021 · 2 comments

Comments

@miliadis
Copy link

miliadis commented Feb 25, 2021

Hi there,

I think there are few issues with vertical words.

Shouldn't this function https://github.com/JaidedAI/EasyOCR/blob/master/easyocr/easyocr.py#L344 output max_width as well in order to update max_width = max(max_width, imgH) in the next line ? It seems that if there is a long vertical word then it's capped by imgH and recognition is usually wrong.

Also, I realized that images are cropped and resized in here https://github.com/JaidedAI/EasyOCR/blob/master/easyocr/easyocr.py#L341 based on their ratio which makes long image crops, that is h >> w, very small (their width is squeezed a lot). Then, these resized images are rotated (90, 180 and 270) in https://github.com/JaidedAI/EasyOCR/blob/master/easyocr/easyocr.py#L344. I think the images should be rotated before they get resized.

@miliadis miliadis changed the title Vertical recogntion Vertical recognition Feb 25, 2021
@johan456789
Copy link

This can be closed now that the PR is merged?

@miliadis
Copy link
Author

miliadis commented Aug 7, 2021

Yes, thank you.

@miliadis miliadis closed this as completed Aug 7, 2021
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

2 participants