`louvain.set_rng_seed` throws error in current release about louvain-igraph HOT 4 CLOSED

Thanks for reporting it! I thought it was already changed previously to work similarly to the leidenalg package. That is, the seed should be set on the Optimiser object (or in find_partition). It seems that this was not properly finished, I'll have a look at it shortly.

from louvain-igraph.

@ivirshup, could you check if this solves your issue? You would have to change this line

https://github.com/theislab/scanpy/blob/127c8b2ce9aecee00d2b42118a1b343e1e51555b/scanpy/tools/_louvain.py#L144

dependent on the version of louvain, and for the new version you should pass it as a parameter to find_partition (or to the Optimiser object). I will make a new proper release 0.7.0, to indicate it is incompatible with version 0.6.x and yank version 0.6.2. You could then either test for the existence of louvain.set_rng_seed or test the version of the package.

from louvain-igraph.

That looks like it'll work.

I'm having trouble checking since I'm not sure how to install a development version of this library. I'm happy to take your word for it though, and checking when there's a new release.

from louvain-igraph.

OK, I will start preparing a release.

FWIW, you can download binary wheels for all platforms from the current CI build, see https://ci.appveyor.com/project/vtraag/louvain-igraph, under "artifacts" for each job. The macOS version is now rebuilding, for some reason that job was terminated.

from louvain-igraph.