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

CuGraph+PyG Wrappers and Loaders #2567

Merged
merged 84 commits into from
Aug 31, 2022

Conversation

alexbarghi-nv
Copy link
Member

@alexbarghi-nv alexbarghi-nv commented Aug 11, 2022

Does not include GaaS. Supports heterogeneous graphs and MG. Includes example notebook. Also includes a required fix to uniform_neighbor_sample to handle graphs without weights.

Closes https://github.com/rapidsai/graph_dl/issues/20

Closes https://github.com/rapidsai/graph_dl/issues/21

Closes #2590

Closes #2438

Closes #2428

@alexbarghi-nv alexbarghi-nv removed the bug Something isn't working label Aug 17, 2022
@alexbarghi-nv alexbarghi-nv marked this pull request as ready for review August 17, 2022 19:29
@alexbarghi-nv alexbarghi-nv requested review from a team as code owners August 17, 2022 19:29
@codecov-commenter
Copy link

codecov-commenter commented Aug 17, 2022

Codecov Report

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

@@               Coverage Diff               @@
##             branch-22.10    #2567   +/-   ##
===============================================
  Coverage                ?   60.19%           
===============================================
  Files                   ?      112           
  Lines                   ?     6142           
  Branches                ?        0           
===============================================
  Hits                    ?     3697           
  Misses                  ?     2445           
  Partials                ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@VibhuJawa VibhuJawa left a comment

Choose a reason for hiding this comment

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

Small suggestion, LGTM otherwise.

@alexbarghi-nv
Copy link
Member Author

rerun tests

@rlratzel
Copy link
Contributor

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 1dc77bb into rapidsai:branch-22.10 Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking change feature request New feature or request
Projects
None yet
5 participants