Skip to content

chore: optimize code #73

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 4 commits into from
Apr 8, 2025
Merged

chore: optimize code #73

merged 4 commits into from
Apr 8, 2025

Conversation

SWHL
Copy link
Member

@SWHL SWHL commented Apr 7, 2025

适配rapidocr 2.0

  • 更新LoadImage,支持传入img url
  • 统一logger,和rapidocr一致
  • 测试python 3.13兼容性,放开numpy <2.0python<3.13限制

@SWHL SWHL requested review from Copilot and Joker1212 April 7, 2025 01:36
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (3)

demo.py:12

  • RapidOCR is used without an explicit import; consider importing RapidOCR from the appropriate module to avoid runtime errors.
ocr_engine = RapidOCR()

demo.py:13

  • The identifier 'VisRes' is undefined; consider replacing it with the correct visualization class (e.g., VisTable) or importing it properly.
vis_ocr = VisRes()

demo.py:51

  • The function or variable 'viser' is not defined; please verify if it should be an instance of VisTable or another defined function.
vis_imged = viser(

@SWHL SWHL requested a review from Copilot April 8, 2025 01:22
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

demo.py:12

  • The identifier 'VisRes' is undefined; if the intended class is 'VisTable', please update the instantiation accordingly.
vis_ocr = VisRes()

@SWHL SWHL self-assigned this Apr 8, 2025
@SWHL SWHL merged commit aa0fd63 into main Apr 8, 2025
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.

1 participant