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

Add closeness_centrality app and test case of duplicated mode modification #336

Merged
merged 17 commits into from
Jun 3, 2021

Conversation

acezen
Copy link
Collaborator

@acezen acezen commented May 21, 2021

What do these changes do?

  • implement closeness_centrality base on duplicated mode of networkx Graph
  • use closeness_centrality to test modification/copy/to_direcrted/to_undirected and subgraph in duplicated mode.
  • add test case of closeness_centrality

Related issue number

#318
#314

@acezen acezen force-pushed the acezen/all_pair_dijkstra_path_length branch 2 times, most recently from faffa02 to bb50133 Compare May 31, 2021 05:26
@acezen acezen changed the title Add all pair dijkstra path length app Add closeness_centrality app and test case of duplicated mode modification May 31, 2021
@acezen acezen force-pushed the acezen/all_pair_dijkstra_path_length branch 2 times, most recently from 83c97da to a54296c Compare June 1, 2021 07:35
@codecov-commenter
Copy link

codecov-commenter commented Jun 1, 2021

Codecov Report

Merging #336 (68434b4) into main (0b7e314) will increase coverage by 0.00%.
The diff coverage is 84.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #336   +/-   ##
=======================================
  Coverage   77.36%   77.36%           
=======================================
  Files          57       57           
  Lines        5301     5302    +1     
=======================================
+ Hits         4101     4102    +1     
  Misses       1200     1200           
Impacted Files Coverage Δ
...ython/graphscope/analytical/app/katz_centrality.py 53.84% <ø> (ø)
python/graphscope/framework/app.py 87.87% <84.00%> (+0.07%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b7e314...68434b4. Read the comment docs.

@acezen acezen marked this pull request as ready for review June 1, 2021 13:52
@acezen acezen force-pushed the acezen/all_pair_dijkstra_path_length branch from 6e75f75 to 1d35990 Compare June 2, 2021 05:00
@acezen acezen force-pushed the acezen/all_pair_dijkstra_path_length branch from c2874a4 to d83e8cc Compare June 2, 2021 09:08
@acezen acezen force-pushed the acezen/all_pair_dijkstra_path_length branch from d83e8cc to ee43167 Compare June 2, 2021 09:12
@acezen acezen merged commit 6a98c7e into alibaba:main Jun 3, 2021
acezen added a commit to acezen/GraphScope that referenced this pull request Jun 4, 2021
@acezen acezen deleted the acezen/all_pair_dijkstra_path_length branch September 3, 2021 10:51
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.

3 participants