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 MG wrapper for HITS #2088

Merged
merged 39 commits into from
Mar 30, 2022
Merged

Conversation

jnke2016
Copy link
Contributor

@jnke2016 jnke2016 commented Feb 20, 2022

This PR wraps the MG C++ implementation of HITS following the pylibcugraph API
Awaiting MG support to the C API

closes #2026

@jnke2016 jnke2016 requested a review from a team as a code owner February 20, 2022 13:55
@rlratzel rlratzel added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change python labels Feb 24, 2022
@rlratzel rlratzel marked this pull request as draft February 24, 2022 16:06
@BradReesWork BradReesWork added this to the 22.04 milestone Mar 17, 2022
@jnke2016 jnke2016 marked this pull request as ready for review March 18, 2022 15:10
@BradReesWork BradReesWork changed the title [skip-ci] Add MG wrapper for HITS Add MG wrapper for HITS Mar 20, 2022
python/cugraph/cugraph/dask/link_analysis/hits.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/dask/link_analysis/hits.py Outdated Show resolved Hide resolved
python/pylibcugraph/pylibcugraph/hits.pyx Show resolved Hide resolved
python/cugraph/cugraph/dask/link_analysis/hits.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/tests/dask/test_mg_hits.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/tests/dask/test_mg_hits.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/tests/dask/test_mg_hits.py Outdated Show resolved Hide resolved
python/cugraph/cugraph/tests/dask/test_mg_hits.py Outdated Show resolved Hide resolved
python/pylibcugraph/pylibcugraph/hits.pyx Outdated Show resolved Hide resolved
python/pylibcugraph/pylibcugraph/hits.pyx Outdated Show resolved Hide resolved
python/pylibcugraph/pylibcugraph/hits.pyx Outdated Show resolved Hide resolved
python/pylibcugraph/pylibcugraph/hits.pyx Outdated Show resolved Hide resolved
@BradReesWork
Copy link
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 5b84e53 into rapidsai:branch-22.04 Mar 30, 2022
rapids-bot bot pushed a commit that referenced this pull request Mar 31, 2022
Closes #2108 when merged. Requires both #2088 and #2156 to be merged before, the former because this uses MGGraph, and the later because of the C implementation of neighborhood sampling.

Authors:
  - https://github.com/betochimas
  - Joseph Nke (https://github.com/jnke2016)
  - Rick Ratzel (https://github.com/rlratzel)

Approvers:
  - Don Acosta (https://github.com/acostadon)
  - Rick Ratzel (https://github.com/rlratzel)
  - Joseph Nke (https://github.com/jnke2016)
  - Chuck Hastings (https://github.com/ChuckHastings)
  - Jordan Jacobelli (https://github.com/Ethyling)

URL: #2118
@jnke2016 jnke2016 deleted the fea-mg_hits branch September 24, 2022 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ENH] Add MG HITS to the cugraph Python API
7 participants