Skip to content
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

Update seal_recognition.py #2491

Open
wants to merge 2 commits into
base: release/3.0-beta1
Choose a base branch
from

Conversation

BUJIQI
Copy link

@BUJIQI BUJIQI commented Nov 15, 2024

原代码由于设定了list[0],导致只能读input变量下第一张图片,使得当input为list或文件夹时,只处理第一张图片,未实现readme所写批处理功能
现代码通过使用列表推导式,对此问题进行了优化改进

此为PR说明模版

如提交的为部署C++代码,请确认以下自测点是否完成(完成勾选即可),并保留以下的自测点列表,便于Reviewer知晓。

  • [√ ] Linux下测试通过
    • [ √] batch = 1 预测
    • batch > 1预测
    • 多GPU卡预测
    • TensorRT预测
    • Triton预测
  • Windows下测试通过
    • batch = 1预测
    • batch > 1预测

Copy link

paddle-bot bot commented Nov 15, 2024

Thanks for your contribution!

@CLAassistant
Copy link

CLAassistant commented Nov 15, 2024

CLA assistant check
All committers have signed the CLA.

@Sunting78
Copy link
Collaborator

  1. 执行该PR修改后的代码,保存文件会报错。
  2. 目前产线支持pdf多页,如果传入list无法正确区分页码pageid,需要更正这个逻辑。

@BUJIQI
Copy link
Author

BUJIQI commented Nov 19, 2024

  1. 执行该PR修改后的代码,保存文件会报错。
  2. 目前产线支持pdf多页,如果传入list无法正确区分页码pageid,需要更正这个逻辑。
  1. 执行该PR修改后的代码,保存文件会报错。
  2. 目前产线支持pdf多页,如果传入list无法正确区分页码pageid,需要更正这个逻辑。

重新修改了一下逻辑,在我本地测试了可以传入list和处理多页pdf了qvq

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants