Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Improve Graph Rendering #15

Closed
RamonMeffert opened this issue Nov 7, 2020 · 5 comments
Closed

Improve Graph Rendering #15

RamonMeffert opened this issue Nov 7, 2020 · 5 comments
Assignees
Labels
bug Something isn't working gossip graph high prio

Comments

@RamonMeffert
Copy link
Owner

  • Consider rendering the first N agents in a circle to improve quality of rendered graph
  • Look at how GraphViz renders graph (probably very hard)
  • At the very least, make sure nodes do not overlap
@RamonMeffert RamonMeffert added bug Something isn't working gossip graph labels Nov 7, 2020
@RamonMeffert RamonMeffert added this to To Do in Tools for Gossip Nov 7, 2020
@RamonMeffert RamonMeffert self-assigned this Nov 8, 2020
@RamonMeffert
Copy link
Owner Author

This should also take into account the reworked graph data structure (#28), as currently edges can overlap (I think)

@RamonMeffert
Copy link
Owner Author

A possible improvement for this might be to use Graph.update when a change to the graph is made. I have a feeling this might keep nodes in the same positions. Not sure though

@m4lvin
Copy link
Collaborator

m4lvin commented Feb 3, 2021

Some examples that are currently rendered with overlapping nodes:

  • Ab B bCd D dE (a "double bush")
  • Ab B bC D E F (some disconnected nodes)
  • A B abC abD abE abF (some nodes with the same links)

@RamonMeffert
Copy link
Owner Author

The library I'm using to render the graph, elm-visualization, just received a major update. Notably, they've added support for collision detection in force-directed graphs. This will solve this problem! I'll update to this new version when I find some time, hopefully soon.

@RamonMeffert
Copy link
Owner Author

Should be better now!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working gossip graph high prio
Projects
No open projects
Development

No branches or pull requests

2 participants