Skip to content

🐛 update Use this model snippets for PaddleOCR models #1645

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

Merged
merged 1 commit into from
Jul 23, 2025

Conversation

TingquanGao
Copy link
Contributor

@TingquanGao TingquanGao commented Jul 22, 2025

Hi HuggingFace team,

Thank you very much for supporting the PaddleOCR snippets. However, the code required for PaddleOCR models differs depending on the specific task. To address this, I have updated the model tags to include task information. The snippets will also need to be updated accordingly.

Please review this PR, and feel free to let me know if any modifications are needed.

Thank you for your attention and support.

@TingquanGao TingquanGao force-pushed the fix_ppocr branch 3 times, most recently from 476de54 to 35c6fdb Compare July 22, 2025 12:50
@TingquanGao TingquanGao changed the title 🐛 update Use this model snippets for PaddleOCR's models 🐛 update Use this model snippets for PaddleOCR models Jul 22, 2025
Copy link
Member

@Vaibhavs10 Vaibhavs10 left a comment

Choose a reason for hiding this comment

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

Nicely done! looked through the docs to compare snippets a bit, they look good.

Copy link
Member

@Vaibhavs10 Vaibhavs10 left a comment

Choose a reason for hiding this comment

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

Please do remember to lint so that the tests are happy!

@TingquanGao
Copy link
Contributor Author

Thank you for your feedback! I’ve formatted the code. Please take another look.

Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

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

Thank you!

@Wauplin Wauplin merged commit 634102f into huggingface:main Jul 23, 2025
2 of 4 checks passed
@Wauplin
Copy link
Contributor

Wauplin commented Jul 23, 2025

Thanks a lot @TingquanGao not only for this PR but also to make PaddleOCR very well integrated with the Hub! 🤗

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