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

1、修复assert_ocr_not_exist断言Bug2、扩展assert_ocr_exist 与 assert_ocr_not_exist断言3、优化OCR识别范围筛选方法 #96

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

Marszzz1116
Copy link

1、修复assert_ocr_not_exist断言Bug
修复在传入多个参数时,断言无效的Bug

2、扩展assert_ocr_exist 与 assert_ocr_not_exist断言
增加bbox参数,新增区域断言支持,主要用于解决当整个屏幕内容较多,干扰较大时OCR精准度过低问题

3、优化OCR识别范围筛选方法
修改了使用方法,历史调用中在区域内查找到了多个关键字会返回第一组坐标,但该结果违背了使用此方法的初衷(当结果出现多个关键词坐标时,进行范围筛选,得出我们需要的唯一坐标),这里增加了过程校验,确保最终只能返回唯一坐标

Copy link

github-actions bot commented Aug 2, 2024

CLA Assistant Lite bot:
提交邮箱中包含我们的合作伙伴,但您似乎并非合作伙伴的成员或对接人,请联系相关对接人将您添加至组织之中,或由其重新发起 Pull Request。
The commit email domain belongs to one of our partners, but it seems you are not yet a member of the current organization, please contact the contact person to add you to the organization or let them submit the Pull Request.

huanghaizhen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Marszzz1116

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@deepin-ci-robot
Copy link

Hi @Marszzz1116. Thanks for your PR.

I'm waiting for a linuxdeepin member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mikigo mikigo merged commit f35bb30 into linuxdeepin:master Aug 2, 2024
0 of 2 checks passed
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