Skip to content

Enhance Multilingual Support with Chinese and German Label Recognition #447

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

Closed
wants to merge 8 commits into from

Conversation

neozhu
Copy link
Contributor

@neozhu neozhu commented Mar 14, 2024

Description:

This Pull Request introduces significant enhancements to our application's language recognition capabilities. With the integration of new features, we're now extending our support to include both Chinese and German languages, catering to a broader user base and improving our application's versatility in multilingual environments.

Key Features and Changes:

  1. Chinese Label Recognition with PaddleOCR: We've implemented Chinese label recognition utilizing the powerful PaddleOCR framework. This addition enables our application to accurately recognize and process labels in Chinese, facilitating users in Chinese-speaking regions to interact with our application more effectively.

  2. German and Chinese Language Support: Building on our commitment to support multiple languages, we've added recognition capabilities for German alongside Chinese. This ensures that users from German-speaking countries can also benefit from our application's features without language barriers.

Benefits:

  • Broader Reach: By supporting more languages, our application becomes accessible to a wider audience, enhancing user experience and satisfaction.
  • Improved Accuracy: The use of PaddleOCR for Chinese label recognition promises high accuracy and efficiency, ensuring that our application meets the needs of users requiring Chinese language support.
  • Enhanced User Experience: Multilingual support significantly improves the overall user experience, allowing users to interact with the application in their preferred language.

image

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.03%. Comparing base (b0401d2) to head (c85b0e8).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #447   +/-   ##
=======================================
  Coverage   47.03%   47.03%           
=======================================
  Files          13       13           
  Lines         640      640           
=======================================
  Hits          301      301           
  Misses        339      339           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@niklub niklub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the great contribution! A couple of comments left to address, and then we can merge it to the master

@neozhu
Copy link
Contributor Author

neozhu commented Mar 18, 2024

Thanks for the great contribution! A couple of comments left to address, and then we can merge it to the master

Thank you for the review and the feedback. I appreciate your suggestions and will make the necessary changes right away.

@neozhu
Copy link
Contributor Author

neozhu commented Mar 20, 2024

I've made the necessary changes and have committed it again. Could you please review it once more?

@niklub
Copy link
Contributor

niklub commented Mar 26, 2024

Could you please separate the changes for paddleocr and tesseract into two different PRs ? That would simplify the review process and conflict resolution. Thanks!

@neozhu
Copy link
Contributor Author

neozhu commented Mar 27, 2024

I'm closing this PR and have already resubmitted two PRs, #466 and #465 . Please review them when you have time. Thank you.

@neozhu neozhu closed this Mar 27, 2024
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

Successfully merging this pull request may close these issues.

3 participants