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.

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

Stochastic zero row + pagerank #1001

Merged
merged 7 commits into from Nov 6, 2013
Merged

Conversation

hagberg
Copy link
Member

@hagberg hagberg commented Nov 5, 2013

Allow zero out-degree (row sum) but warn in stochastic_graph
Allow (working) multigraph input to pagerank_numpy and pagerank_scipy
Clarify types of graphs allowed

@ghost ghost assigned hagberg Nov 5, 2013
Match convergence criteria to be the same for pagerank and pagerank_scipy
Now pagerank_scipy() is the same as pagerank().
That is, both will possibly produce the same incorrect answer if you put in negative weights.
hagberg added a commit that referenced this pull request Nov 6, 2013
Stochastic zero row + pagerank

The various pagerank(), pagerank_numpy(), and pagerank_scipy() should produce the same results now for cases with weighted graphs with zero row sums or negative weights.   The negative weight case may produce answers but I don't know if they are meaningful (certainly that breaks the intentions and assumptions of the algorithm.)  The convergence tolerances now match between implementations too.
@hagberg hagberg merged commit efec7a6 into networkx:master Nov 6, 2013
@hagberg hagberg deleted the stochastic-zero branch December 22, 2013 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant