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 infer_demo.py #8820

Open
wants to merge 1 commit into
base: release/2.7
Choose a base branch
from

Conversation

HeShiwen
Copy link

修复ONNXRuntime部署示例颜色错误

Bug组件 Bug Component
Deploy

Bug描述 Describe the Bug
PaddleDetection\deploy\third_engine\demo_onnxruntime\infer_demo.py进行ONNX推理,得到的图片结果颜色错误。原因是在保存图片之前没有将RGB转为OpenCV默认的颜色格式,应该在174与175行之间添加以下代码:srcimg = cv2.cvtColor(srcimg, cv2.COLOR_RGB2BGR)

复现环境 Environment
PaddleDetection2.7

Fix the bug of result image color issue
Copy link

paddle-bot bot commented Feb 19, 2024

Thanks for your contribution!

@CLAassistant
Copy link

CLAassistant commented Feb 19, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@lyuwenyu lyuwenyu left a comment

Choose a reason for hiding this comment

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

LGTM

@lyuwenyu
Copy link
Collaborator

辛苦develop分支也修改下~

@HeShiwen
Copy link
Author

我刚接触Github,还在学习。
请问合并到develop分支是否需要发起另外一个pr?
这个pr是否符合要求?还需要做什么,还是说等待合并就可以了呢

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.

None yet

3 participants