search

GuyAllard / markov_clustering

markov clustering in python
MIT License
170 stars 37 forks source link

Two issues: "nx.to_scipy_sparse_matrix" not exist and the inflation hyperparameter not working? #37

Open HsiangHung opened 1 year ago

HsiangHung commented 1 year ago

Hi MCL Community,

I found two issues:

  1. In the sample code, nx.to_scipy_sparse_matrix doesn't exist in networkx 3.0. The document needs to be updated to nx.to_scipy_sparse_array.
  2. I tried the doc: https://markov-clustering.readthedocs.io/en/latest/readme.html and found different hyperparameter inflation =1.5-2.6 also gave the same results and modularity are different from the doc page.

image

Also inflation=1.1 and inflation=2.5 gave the same clustering outcome. I think there is a bug. Can someone check?