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.

Sign up for GitHub

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

Jump to bottom

Update egonet implementation #2874

Merged
merged 76 commits into from
Nov 17, 2022
Merged

Update egonet implementation #2874

merged 76 commits into from
Nov 17, 2022

Conversation

jnke2016
Copy link
Contributor

@jnke2016 jnke2016 commented Nov 2, 2022 •
edited
Loading

There is a CAPI implementation of egonet and this PR:

  1. Adds a PLC implementation of egonet by leveraging the CAPI
  2. Refactor the python implementation by leveraging PLC
  3. Add a python MG implementation of egonet.
  4. Add/update docstrings and tests

closes #2873

@BradReesWork BradReesWork added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Nov 8, 2022
@BradReesWork BradReesWork added this to the 22.12 milestone Nov 8, 2022
@jnke2016 jnke2016 changed the title Update egonet implementation [WIP] Update egonet implementation Nov 14, 2022
@jnke2016 jnke2016 marked this pull request as ready for review November 14, 2022 02:23
@jnke2016 jnke2016 requested review from a team as code owners November 14, 2022 02:23
@jnke2016 jnke2016 changed the title [WIP] Update egonet implementation Update egonet implementation Nov 16, 2022
@jnke2016 jnke2016 requested a review from a team as a code owner November 16, 2022 19:39
rlratzel
rlratzel reviewed Nov 16, 2022
View reviewed changes
cpp/src/c_api/extract_ego.cpp Outdated Show resolved Hide resolved
python/cugraph/cugraph/community/egonet.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/community/egonet.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/community/egonet.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/dask/community/egonet.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/dask/community/egonet.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/dask/community/egonet.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/tests/mg/test_mg_egonet.py Outdated Show resolved Hide resolved
python/pylibcugraph/pylibcugraph/egonet.pyx Outdated Show resolved Hide resolved
python/pylibcugraph/pylibcugraph/egonet.pyx Show resolved Hide resolved
@jnke2016
Copy link
Contributor Author

rerun tests

@ajschmidt8 ajschmidt8 removed the request for review from a team November 17, 2022 15:32
@ChuckHastings
Copy link
Collaborator

@gpucibot merge

@rapids-bot rapids-bot bot merged commit e8085a1 into rapidsai:branch-22.12 Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChuckHastings ChuckHastings ChuckHastings approved these changes

@rlratzel rlratzel rlratzel approved these changes

Assignees
No one assigned
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Milestone
22.12
Development

Successfully merging this pull request may close these issues.

[FEA]: Update python implementation of egonet
6 participants
@jnke2016 @codecov-commenter @ChuckHastings @rlratzel @BradReesWork @galipremsagar