search

ericmjl / Network-Analysis-Made-Simple

An introduction to network analysis and applied graph theory using Python and NetworkX
https://ericmjl.github.io/Network-Analysis-Made-Simple/index.html
MIT License
1.03k stars 401 forks source link

python-louvain issues #46

Closed ericmjl closed 6 years ago

ericmjl commented 6 years ago

@MridulS can you please build a conda-forge package for python-louvain? Its presence as a pip-installed package is causing some issues.

ericmjl commented 6 years ago

I've started the PR: https://github.com/conda-forge/staged-recipes/pull/5797

ericmjl commented 6 years ago

Btw, I just checked: python-louvain requires networkx < 2.0.0. This tutorial is about teaching networkx >= 2.0.0. This is going to be an issue. Is Louvain absolutely required, or can you use a different community detection method?

MridulS commented 6 years ago

They updated it last year. Does it show up while conda install?

python-louvain>=0.9 works with networkx>=2.0

I'm using with networkx==2.1 and python-louvain=0.10

On Sat, May 5, 2018 at 6:36 PM, Eric Ma notifications@github.com wrote:

Btw, I just checked: python-louvain requires networkx < 2.0.0. This tutorial is about teaching networkx >= 2.0.0. This is going to be an issue. Is Louvain absolutely required, or can you use a different community detection method?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ericmjl/Network-Analysis-Made-Simple/issues/46#issuecomment-386817971, or mute the thread https://github.com/notifications/unsubscribe-auth/AFHYlDwO6rbdy-Q6LuOX-WgiSjVG0FTMks5tvdUFgaJpZM4TzoQS .

ericmjl commented 6 years ago

Interesting: what about the GitHub repo then? https://github.com/taynaud/python-louvain/blob/master/requirements.txt

ericmjl commented 6 years ago

I guess everything should be fine, now that the python-louvain package is up on conda-forge. Closing this issue.