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

Added closeness centrality #1325

Closed
wants to merge 2 commits into from
Closed

Conversation

miratepuffin
Copy link
Collaborator

What changes were proposed in this pull request?

  • Updated dijkstra's be able to run in any direction and without weights

Why are the changes needed?

Algorithms was missing

Does this PR introduce any user-facing change? If yes is this documented?

Yes and documented

How was this patch tested?

All changes to dijkstras and closeness have new tests

Issues

Are there any further changes required?

Copy link
Contributor

@Haaroon Haaroon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

requires docs & fix formatting

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants