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

Use BuchheimWalkerConfiguration to show tree merge node has problem #8

Closed
zhangxinagjunHJH opened this issue Sep 24, 2020 · 5 comments

Comments

@zhangxinagjunHJH
Copy link

zhangxinagjunHJH commented Sep 24, 2020

I use BuchheimWalkerConfiguration like this, there are 2 branch and one merge node, it's OK

`graph.addEdge(node1, node2, paint: Paint()..color = Colors.red);
graph.addEdge(node2, node6);
graph.addEdge(node6, node5);

graph.addEdge(node2, node3);
graph.addEdge(node3, node4);
graph.addEdge(node4, node5);`
image

but if i change the order like this, it will show wrong connect line

`graph.addEdge(node1, node2, paint: Paint()..color = Colors.red);
graph.addEdge(node2, node3);
graph.addEdge(node3, node4);
graph.addEdge(node4, node5);

graph.addEdge(node2, node6);
graph.addEdge(node6, node5);`
image

@nabil6391
Copy link
Owner

Basically BuchheimWalker is for treelayouts only, if you want merged nodes, suggest you use Sugiyama

@zhangxinagjunHJH
Copy link
Author

@nabil6391 Yes, I checked code both Sugiyama and FruchtermanReingoldAlgorithm, but i found they don't support change orientation to horizontal. can add this feature to them?

@nabil6391
Copy link
Owner

Hm that is correct, yes I would prefer to add them but do not think it can be done soon.

@zhangxinagjunHJH
Copy link
Author

@nabil6391 OK, Thank you very much! I will continue to pay attention and wait.

@nabil6391
Copy link
Owner

Added support for different orientation in 0.6.5. Try it out. Closing for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants