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

ONNX: Reverse order of inputs for Concat in YOLOV5s #1012

Closed
NKSagarReddy opened this issue Nov 30, 2022 · 3 comments
Closed

ONNX: Reverse order of inputs for Concat in YOLOV5s #1012

NKSagarReddy opened this issue Nov 30, 2022 · 3 comments
Assignees
@NKSagarReddy
Labels
bug no repro

Comments

@NKSagarReddy
Copy link

NKSagarReddy commented Nov 30, 2022 •
edited
Loading

  • Netron app (website) : https://netron.app/
  • OS and browser version: macOS 12.5 (21G72) + Chrome Version 106.0.5249.119

Steps to Reproduce:

  1. Upload yolov5s.onnx in netron.app
  2. Concat Modules - [12, 16, 19, 22] are being given inputs in reverse order. (Correct order(Tensorboard) is in https://medium.com/analytics-vidhya/object-detection-algorithm-yolo-v5-architecture-89e0a35472ef)
@lutzroeder
Copy link
Owner

lutzroeder commented Dec 1, 2022 •
edited
Loading

@NKSagarReddy can you please share the ONNX file and a screenshot or example?

@lutzroeder lutzroeder closed this as not planned Won't fix, can't repro, duplicate, stale Dec 1, 2022
@lutzroeder lutzroeder changed the title Visualizing reverse order of inputs for some Concat modules in YOLOV5s ONNX: Reverse order of inputs for Concat in YOLOV5s Dec 1, 2022
@NKSagarReddy
Copy link
Author

@lutzroeder
Yolov5s.onnx - https://drive.google.com/file/d/1-99xFLF2v3jz_PgS5mJI5rI9k4pkjf3i/view?usp=sharing
yolov5s in netron
The Concat after model.18.conv is model.19
For the model.19 concat layer, the model.18.conv output should be the first argument, not the second.
Expected representation:
image

@lutzroeder
Copy link
Owner

The order is listed in properties. The graph layout library does not support edge ordering. dagrejs/dagre#112

screenshot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NKSagarReddy NKSagarReddy

Labels
bug no repro
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lutzroeder @NKSagarReddy