Walktrap: merges matrix is necessary for membership calculation, but isn't checked for or documented

Bug #1066072 reported by Christopher Lu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
igraph
Fix Released
Medium
Tamás Nepusz

Bug Description

The merges matrix must be non-NULL for walktrap to report vertex memberships at the end, but this isn't specified in the documentation, nor does the code check for it.

A patch against top of trunk is included.

Tags: walktrap
Revision history for this message
Christopher Lu (9e9o1ko8b2f5xpiibgscjzlhu-chris-0zxvj9hhx1hzo5xiyhxz186cr) wrote :
Tamás Nepusz (ntamas)
Changed in igraph:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Tamás Nepusz (ntamas)
milestone: none → 0.6.1
Revision history for this message
Tamás Nepusz (ntamas) wrote :

Merged in #3008 in trunk and #2970 in 0.6-main, thanks!

Changed in igraph:
status: In Progress → Fix Committed
Changed in igraph:
status: Fix Committed → Fix Released
Revision history for this message
Gábor Csárdi (gabor.csardi) wrote : Continue on github

The development of igraph has moved to github, so please do not comment on this bug here. You are of course welcome to comment on github, here:
https://github.com/igraph/igraph/issues/241

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.