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

[SPARK-9975][GraphX] Add Normalized Closeness Centrality to Spark GraphX #8195

Closed
wants to merge 1 commit into from

Conversation

kbastani
Copy link

  • Added normalized closeness centrality to GraphX library
  • Added closeness centrality test suite
  • Added closeness centrality operation to GraphOps

Links to JIRA issue: https://issues.apache.org/jira/browse/SPARK-9975

* Added normalized closeness centrality to GraphX library
* Added closeness centrality test suite
* Added closeness centrality operation to GraphOps
@kbastani kbastani changed the title [SPARK-9975] Add Normalized Closeness Centrality to Spark GraphX [SPARK-9975][GraphX] Add Normalized Closeness Centrality to Spark GraphX Aug 14, 2015
@xdev-developer
Copy link

I might be wrong but I think centrality calculation produce wrong results.
Example graph:

Vertices: A, B, C, D, E, F
Edges: (A, B), (A, C), (A, D), (A, E), (A, F)

Calculation result is:

C closeness_centrality: 0.0
B closeness_centrality: 0.0
E closeness_centrality: 0.0
A closeness_centrality: 1.2
F closeness_centrality: 0.0
D closeness_centrality: 0.0

result of A closeness_centrality should be 1.0

@andrewor14
Copy link
Contributor

ok to test. @kbastani can you address the comments? Also maybe this is of interest to @ankurdave.

@SparkQA
Copy link

SparkQA commented Dec 15, 2015

Test build #47750 has finished for PR 8195 at commit 3e900b1.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@skonto
Copy link
Contributor

skonto commented Feb 1, 2016

i will have a look at this.

@srowen
Copy link
Member

srowen commented Feb 2, 2016

I doubt this will be merged at this stage given lack of graphx activity. Instead I think this should be closed

@asfgit asfgit closed this in 6acc72a Apr 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants