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

K-core implementation for undirected graphs #1933

Merged
merged 51 commits into from
Nov 12, 2021

Conversation

seunghwak
Copy link
Contributor

@seunghwak seunghwak commented Nov 8, 2021

Close #1909.

This PR aims to implement K-core only for undirected graphs.

  • Implement K-core implementation
  • Test/debug on SG
  • Test/debug on MG

@seunghwak seunghwak added feature request New feature or request 2 - In Progress non-breaking Non-breaking change labels Nov 8, 2021
@seunghwak seunghwak added this to the 21.12 milestone Nov 8, 2021
@seunghwak seunghwak self-assigned this Nov 8, 2021
@seunghwak seunghwak changed the title [WIP][skip-ci] K-core implementation for undirected graphs K-core implementation for undirected graphs Nov 9, 2021
divyegala and others added 3 commits November 9, 2021 10:52
…ers to k_last for the vertices that belong to k_last+1-core as this information is useful in triangle counting so no need to throw away useful information
@codecov-commenter
Copy link

codecov-commenter commented Nov 11, 2021

Codecov Report

❗ No coverage uploaded for pull request base (branch-21.12@fda2466). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 3ae6d9d differs from pull request most recent head 5bce534. Consider uploading reports for the commit 5bce534 to get more accurate results
Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.12    #1933   +/-   ##
===============================================
  Coverage                ?   70.35%           
===============================================
  Files                   ?      143           
  Lines                   ?     8903           
  Branches                ?        0           
===============================================
  Hits                    ?     6264           
  Misses                  ?     2639           
  Partials                ?        0           

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 fda2466...5bce534. Read the comment docs.

Copy link
Collaborator

@ChuckHastings ChuckHastings left a comment

Choose a reason for hiding this comment

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

One observation, willing to merge without the change.

if (graph_view.is_multigraph()) {
CUGRAPH_FAIL("unimplemented."); // check for multi-edges
}
CUGRAPH_FAIL("unimplemented."); // check for self-loops
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we want to add the calls just implemented in #1939 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I should.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@ChuckHastings
Copy link
Collaborator

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 5db6636 into rapidsai:branch-21.12 Nov 12, 2021
@seunghwak seunghwak deleted the fea_k_core_impl branch January 27, 2022 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEA] MNMG K-core
6 participants