CiSE vs fCoSE usage with groups

[ugurdogrusoz]

Currently we make use of the CiSE layout only when we apply grouping (e.g. by Louvain clustering) and the user setting is Group by Circles. We should however consistently use CiSE when this setting is Circles during other operations as well. So:

• On Edit > Add Group for Selected, let's create clustering and apply incremental CiSE if layout is to be applied.

• Wherever we're to apply layout we should check whether there are any groups in the graph (compounds or circles) and if the grouping exists and the setting is by circles, we should apply CiSE instead of fCoSE.

[canbax]

If there is no cluster or just one small cluster I observed that cise works a bit laggy.

In the below image you can see that there is only one small cluster. In the bottom left, you can see that some javascript takes like 1.5 seconds which occurs whenever I run cise layout. If there are lots of clusters I didn't observe that lag.

[ugurdogrusoz]

• 1. Changing "Group using" choice has no effect on the use of compounds vs. clusters
• 2. When Circles are in use and I select a group of nodes, some of which are already in a cluster, others not, we should put all of them into a newly created cluster. Currently, those already in a cluster stay in their cluster and we create a new cluster for the others.

[ugurdogrusoz]

1. Changing "Group using" from Compound to Circle works fine but not the other way around.

[ugurdogrusoz]

• 3. I also noticed that "Perform Layout" will not be incremental when using Circles / CiSE (Recalculates Layout).