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

fix the formatting error #277

Merged
merged 1 commit into from
May 11, 2023
Merged

fix the formatting error #277

merged 1 commit into from
May 11, 2023

Conversation

zhtmike
Copy link
Collaborator

@zhtmike zhtmike commented May 11, 2023

Somehow cur_lr will be [0.111] instead of 0.111 in some cases. Which will raise the formatting error in the logging message from PR #272. This is a fix

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

Motivation

(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)

@SamitHuang SamitHuang merged commit 38fc989 into mindspore-lab:main May 11, 2023
@zhtmike zhtmike deleted the fix_msg branch May 15, 2023 10:29
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