Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Closed
gaborcsardi opened this issue May 4, 2013 · 2 comments
Closed

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

gaborcsardi opened this issue May 4, 2013 · 2 comments
Assignees
@ntamas
Milestone
0.6.4

Comments

@gaborcsardi
Copy link
Contributor

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.

Imported from Launchpad using lp2gh.
@gaborcsardi
Copy link
Contributor Author

(by 9e9o1ko8b2f5xpiibgscjzlhu-chris-0zxvj9hhx1hzo5xiyhxz186cr)

@gaborcsardi
Copy link
Contributor Author

(by ntamas)
Merged in #3008 in trunk and #2970 in 0.6-main, thanks!

@ghost ghost assigned ntamas May 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ntamas ntamas

Labels
None yet
Projects
None yet
Milestone
0.6.4
Development

No branches or pull requests
2 participants
@ntamas @gaborcsardi