search

jaegertracing / jaeger-ui

Web UI for Jaeger
http://jaegertracing.io/
Apache License 2.0
1.15k stars 485 forks source link

[Feature]: upgrade graph layout library #1081

Open yurishkuro opened 2 years ago

yurishkuro commented 2 years ago

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

chethan27 commented 1 year ago

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

wthrajat commented 1 year ago

@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?

yurishkuro commented 1 year ago

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.