-
Notifications
You must be signed in to change notification settings - Fork 13
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
Take node labels into account #28
Comments
This feature should first added to avsdf-base because CiSE uses avsdf-base as its first step to construct cluster circles. Cluster circles should take node labels into account during their construction. |
I'm also interested in this feature, were I to try to develop the necessary feature in https://github.com/iVis-at-Bilkent/avsdf-base should I do it against |
We accept PRs to the unstable branch |
@smarek I'm not exactly sure, but only changing this line and this line in a way that the value of the |
@hasanbalci that'd be of course ideal solution, but in your initial assessment you told @ugurdogrusoz that this must be done in avsdf-base, so i did not consider any alternatives. You're probably best man to do that, will you please do the mentioned change or shall I try to do PR? |
@smarek I'll give it a try. |
Thank you, i've just tried bundle with cise from develop branch, and it performs way better then stable/master relase and indeed the rendering now seems to take labels into account. For example you can see bigger corporate structure (with complex styled node labels) here: https://non.certifix.eu/cz/structure/219 Only thing I'm currently not able to fix is the cluster separation/gravity/springCoeff balance, even with high If anybody'd want to toy with this, changing the params and re-running the layout is as simple as this (in chrome devel console for example) certifix_cytoscape.ciseLayout.nodeDimensionsIncludeLabels = false;
// or assign complete configuration object to "certifix_cytoscape.ciseLayout"
certifix_cytoscape.runLayout(true); |
@smarek Looking at your corporate structure application, I don't believe you're using CiSE properly. This algorithm was devised to visualize cluster/group structures (which you should specify using the |
@ugurdogrusoz thank you for quick reaction clusters are actually assembled dynamically per data in nodes (see What we're actually trying to achieve are clusters that look like left-most (dark blue) or top-most (light blue) in this example: https://blog.js.cytoscape.org/public/images/layouts/cise.png however with regard to node label dimensions. The "hierarchy trees" actually grow quite nice to our expectations, not so the overall layout (separation between the clusters) So we can't want actual "circles" but somewhat tree hierarchy of those clusters, because corporate ownership structures/data are always hierarchical (and sometimes circular or intertwined as well) Another example data (and you can switch between
Do you think it's achievable with CiSE or do we need to switch to different clusters-aware layout? |
CiSE should have an option (see
nodeDimensionsIncludeLabels
in fcose layout for instance) that takes node labels into account when calculating node dimensions.The text was updated successfully, but these errors were encountered: