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 yolov8.cpp larger object boundingbox error #5641

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

caoshihao996
Copy link

Yolov8 has three different heads and output different size feature maps. So the part of boundingboxs needs to be adjusted accordingly.

Yolov8 has three different heads and output different size feature maps.  So the part of boundingboxs needs to be adjusted accordingly.
@tencent-adm
Copy link
Member

tencent-adm commented Aug 19, 2024

CLA assistant check
All committers have signed the CLA.

examples/yolov8.cpp Outdated Show resolved Hide resolved
examples/yolov8.cpp Outdated Show resolved Hide resolved
@whyb
Copy link
Contributor

whyb commented Aug 20, 2024

@caoshihao996 @nihui
并没有从yolov8的ultralytics官方ort推理示例中寻找到类似您pr中所提供的根据num_anchors(或称strideNum)的值针对boundingboxs进行坐标和宽高的调整之代码,只有针对fp16和fp32的处理,如果有更多其他可供参考的文档或者Torch Yolov8后处理的代码,欢迎提出来共同学习讨论~

yolov8的ultralytics官方ort推理示例源码:
https://github.com/ultralytics/ultralytics/blob/main/examples/YOLOv8-ONNXRuntime-CPP/inference.cpp#L263

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants