[quangdungluong]

Summary

Use numpy.asarray to convert rendered PIL crops in RapidOCR, EasyOCR, and Nemotron OCR. This avoids the explicit owning-array copy made by numpy.array at the crop boundary.

Validation

• RapidOCR, EasyOCR and Nemotron tests: passed.
• Ruff and formatting checks passed.

[github-actions]

✅ DCO Check Passed

Thanks @quangdungluong, all your commits are properly signed off. 🎉

[mergify]

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

• title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

[github-actions]

AI triage (advisory)

Checked commit 7f9c724489. The analysis read the diff only. It did not run code from this pull request.

The PR replaces numpy.array with numpy.asarray when it converts PIL crops in the RapidOCR, EasyOCR and Nemotron OCR models. The aim is to avoid an extra copy of each crop.

Topic labels added: performance, ocr.

CI safety: no concern found. A maintainer must still approve CI.

1 related pull request(s) in the same code

PR	State	Signal	Assessment
#4709	open	changes the same code	related: Both PRs change the same numpy.array(high_res_image) line in rapid_ocr_model.py. PR 4709 adds page orientation detection and moves that call into a new _read method. The two PRs do different things, but they will conflict when merged.

_{The ai:* labels are advisory. They do not approve CI and they do not add tests:full. A new push runs this check again.}

[github-actions]

AI first review (advisory): no blocker found

This review covers the full diff at 7f9c724489. It read the code and did not run it.

The change swaps numpy.array for numpy.asarray in three OCR models. A PIL image exports its pixels as bytes, so the new array may be read-only. The three OCR libraries may write to the array or hand it to torch.from_numpy. I could not read their source here to check.

_{A maintainer makes the merge decision. This review never approves the pull request.}

[dolfim-ibm]

lgtm

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!