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
Graph visualization unnecessarily wide #864
Comments
I was just about to report this as well! For example, yolov5n.onnx looks like this for me: This width is somewhat justifiable since the final layers of the model do indeed need the width, but there is no reason for the connections that go from a conv to the following mul to go all the way to the right and back. |
@leosh64 @KarelPeeters @AlessandroMondin can you investigate this and suggest a fix. dagrejs/dagre#239 has some context. The paper describing the algorithm can be found here and the code here. |
It seems the second pass (link) of the Tested with yolov5n.onnx.
So my bet is everyone should edit the code to adjust the view when needed. |
For some networks, the visualization of the graph becomes unnecessarily wide. This makes it difficult to understand connections when zoomed in:
Throughout the model, there is no section that would justify such a wide visualization.
Below some screenshots when zoomed out:
The same issue (unnecessarily high) occurs in horizontal mode:
The text was updated successfully, but these errors were encountered: