jaeger-ui icon indicating copy to clipboard operation
jaeger-ui copied to clipboard

[Feature]: upgrade graph layout library

Open yurishkuro opened this issue 3 years ago • 3 comments

Jaeger depends on a number of deprecated graph layout libs, like dagre and viz.js. It also provides a confusing assortment of graph views (force directed, DAG, and the latest graphs via plexus).

To avoid getting blocked on very old dependencies, we need to move to something newer.

One option to investigate: https://github.com/erikbrinkman/d3-dag

yurishkuro avatar Nov 25 '22 00:11 yurishkuro

@yurishkuro can u assign this to me ? I would like to work on this .

chethan27 avatar Jan 14 '23 16:01 chethan27

@yurishkuro I understand the need to streamline and update the graph visualization dependencies but I wanted to seek clarification on what exactly you mean by "confusing". Are you referring to the presence of multiple visualization choices? Do you prefer a default layout for the graph view? Like if we shift to d3-dag then the default layout would be directed acyclic graph and that would mean to not have the support of, say, force-directed layout anymore (I have not much researched about d3-dag yet but I think we have d3-force for force-directed, for example) Did I get this right?

wthrajat avatar Aug 12 '23 10:08 wthrajat

what exactly you mean by "confusing".

There is no good explanation in the UI why there are two graphs for overall topology. Practically, this was done more for technical reasons, but that doesn't need to be exposed to the user.

yurishkuro avatar Aug 12 '23 19:08 yurishkuro