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

depinject graph visualization improvements #12454

Closed
kocubinski opened this issue Jul 5, 2022 · 5 comments
Closed

depinject graph visualization improvements #12454

kocubinski opened this issue Jul 5, 2022 · 5 comments
Labels
C:depinject Issues and PR related to depinject

Comments

@kocubinski
Copy link
Member

Problem Definition

As mentioned in #11910 we'd like the decisions made by depinject as to explicit and implicit interface binding to be noted in the generated graphviz.

A different visualization algorithm may also be required for non-trivial dependency graphs (e.g. SimApp) for the visualization to be useful.

Proposal

Add interface binding data to graphviz output. Re-evaluate visualization algorithm or possibly tool.

@aaronc
Copy link
Member

aaronc commented Jul 5, 2022

Do you have any alternative visualization algorithms in mind?

@kocubinski
Copy link
Member Author

Nothing specific right now. The Graphviz/dot implementation seems to be Sugiyama-style graph drawing.

@aaronc
Copy link
Member

aaronc commented Jul 5, 2022

Do you notice any specific problems which you think could be solved by a better layout algorithm?

@kocubinski
Copy link
Member Author

kocubinski commented Jul 5, 2022

The amount of edge overlap, the length of rendered edges, size (width) of the graph makes it difficult to follow edges between nodes and at the same time read a node label.

@julienrbrt
Copy link
Member

Closing this as won't do, as no other user has complained about it (are they even looking at the graph :D)

@julienrbrt julienrbrt closed this as not planned Won't fix, can't repro, duplicate, stale Oct 1, 2024
@julienrbrt julienrbrt added the C:depinject Issues and PR related to depinject label Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:depinject Issues and PR related to depinject
Projects
Status: 📋 Backlog
Development

No branches or pull requests

3 participants