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.

Sign up for GitHub

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

Jump to bottom

fix: the result of ocr_cpp_server is not complete #1415

Merged
merged 5 commits into from
Oct 14, 2021
Merged

fix: the result of ocr_cpp_server is not complete #1415

merged 5 commits into from
Oct 14, 2021

Conversation

ShiningZhang
Copy link
Collaborator

@ShiningZhang ShiningZhang commented Oct 13, 2021 •
edited
Loading

  1. 修复示例中ocr_cpp_server结果缺失一行的问题。原因为在general_detection_op中，处理完检测结果将其放入tensor时，将两个检测库放入了两个不同的tensor中。修复为将文字行图片resize到统一尺寸，并且放入一个tensor中，并将tensor_shape的第一维设置为检测框数量。
  2. 对齐了preprocess中与python代码的参数差异
  3. 修改了general_detection_op、preprocess_op文件
  4. 修改了requirement文件，添加pyyaml<6.0，因为release/6.0版本强制要求yaml.load()增加Loader参数

@paddle-bot-old
Copy link

Thanks for your contribution!

@HexToString
Copy link
Collaborator

LGTM

@HexToString HexToString self-requested a review October 14, 2021 02:42
TeslaZhao
TeslaZhao previously approved these changes Oct 14, 2021
View reviewed changes
Copy link
Collaborator

@TeslaZhao TeslaZhao left a comment

Choose a reason for hiding this comment

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

LGTM

TeslaZhao
TeslaZhao approved these changes Oct 14, 2021
View reviewed changes
Copy link
Collaborator

@TeslaZhao TeslaZhao left a comment

Choose a reason for hiding this comment

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

LGTM

@TeslaZhao TeslaZhao merged commit e8133c2 into PaddlePaddle:develop Oct 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TeslaZhao TeslaZhao TeslaZhao approved these changes

@HexToString HexToString Awaiting requested review from HexToString

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ShiningZhang @HexToString @TeslaZhao