Skip to content

fix bug litmit_side_len to master #729

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

Conversation

kk928290341
Copy link
Collaborator

@kk928290341 kk928290341 commented Jul 30, 2024

Thank you for your contribution to the MindOCR repo.
Before submitting this PR, please make sure:

Motivation

修复bug litmit_side_len存在多个参数
(Write your motivation for proposed changes here.)

Test Plan

(How should this PR be tested? Do you require special setup to run the test or repro the fixed bug?)

Related Issues and PRs

(Is this PR part of a group of changes? Link the other relevant PRs and Issues here. Use https://help.github.com/en/articles/closing-issues-using-keywords for help on GitHub syntax)

Copy link
Collaborator

@Mark-ZhouWX Mark-ZhouWX left a comment

Choose a reason for hiding this comment

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

是否应该把line31-37的参数都加进去

@kk928290341
Copy link
Collaborator Author

只有4个是固定的,其他有变量赋值

@Mark-ZhouWX Mark-ZhouWX merged commit 240847f into mindspore-lab:main Jul 30, 2024
2 checks passed
@kk928290341 kk928290341 deleted the fix-bug-limit_side_len-master branch December 5, 2024 06:52
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.

3 participants