Weighted modularity not correct? about louvain-igraph HOT 3 CLOSED

Thanks, I'll look into it! The code is tested on weighted graphs, so in principle it should work. I'm not sure what is going on here, I'll have to take a closer look.

from louvain-igraph.

Initially, I overlooked it, but you are looking at the modularity that is provided by igraph by default. This is indeed an unweighted variant. To get the quality value of the partition, you should call part.quality(), which does provide the corrected weighted modularity. The confusion is understandable, I'll see if I can remove the modularity variable in some way.

from louvain-igraph.

The modularity variable cannot be removed easily, and it would also create an inconsistency with the existing API from igraph, so I am leaving it as is. So, to get the quality of the partition, you always have to call partition.quality(), also when using louvain.ModularityVertexPartition.

from louvain-igraph.