Vincent Traag
Vincent Traag
In particular, the root cause of the problem is that we check whether a neighboring cluster is already added in these conditionals: https://github.com/CWTSLeiden/networkanalysis/blob/a3f342d2c9db52d35c2af2287ff2433e5cb7b093/src/main/java/nl/cwts/networkanalysis/LocalMergingAlgorithm.java#L188 https://github.com/CWTSLeiden/networkanalysis/blob/a3f342d2c9db52d35c2af2287ff2433e5cb7b093/src/main/java/nl/cwts/networkanalysis/FastLocalMovingAlgorithm.java#L162 https://github.com/CWTSLeiden/networkanalysis/blob/a3f342d2c9db52d35c2af2287ff2433e5cb7b093/src/main/java/nl/cwts/networkanalysis/StandardLocalMovingAlgorithm.java#L153 We should probably use a...
No problem, I imagine that you did not check this condition prior to running the algorithm. No, let's leave the issue open, as the program shouldn't crash on this input.
Yes, we would probably do that as well. However, the program still should not crash, even if an edge of zero weight would somehow be included.
> I have a feature request - can you add an additional argument that fixes the number of communities ? Say, if I want 5 communities (even if that results...
This is also an interesting approach. It is doing something else though: it tries to deal with minimum community sizes somehow. This is what is discussed in https://github.com/vtraag/leidenalg/issues/53, with the...
> If K= 10 and M = 5 and the size of the clusters are [10,9,8,6,1,1,1,1,1,1] then using the above approach - is not possible that the new list of...
OK, good to see you got something working that does the trick for you! If you post your code here, it might also help some other people who would like...
Overall, the script looks good to me, thanks for sharing! > if not(aggregate_partition.sizes()[-1] == 0) : > # remove the empty community that exists in the 'middle' > opt.optimise_partition(aggregate_partition) In...
Sorry for the late response @Alex-Mathai-98 ! In principle, the functionality could be interesting for others. However, I would prefer to have an implementation in the C++ layer that could...
I went through all new classes to conform to the code style used throughout this package.