Skip to content

chore: optimize code#73

Merged
SWHL merged 4 commits into
mainfrom
develop
Apr 8, 2025
Merged

chore: optimize code#73
SWHL merged 4 commits into
mainfrom
develop

Conversation

@SWHL

@SWHL SWHL commented Apr 7, 2025

Copy link
Copy Markdown
Member

适配rapidocr 2.0

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

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

Copilot AI left a comment

Copy link
Copy Markdown

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

Copilot AI left a comment

Copy link
Copy Markdown

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.

2 participants