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

MNMG Neighborhood Sampling #2073

Merged
merged 73 commits into from
Mar 17, 2022
Merged

MNMG Neighborhood Sampling #2073

merged 73 commits into from
Mar 17, 2022

Conversation

aschaffer
Copy link
Collaborator

This PR tracks work on MNMG Neighborhood Sampling, for G(C)NN needs.

Dependencies:

  1. [FEA] neighbor sampling in COO/CSR format #1982
  2. Multi gpu sample edges utilities #2064
  3. Integration of https://github.com/rapidsai/cugraph-ops/pull/24 into cugraph

aschaffer and others added 30 commits November 30, 2020 15:00
@aschaffer
Merge pull request #37 from rapidsai/branch-0.17
acf15bc 
Merge latest release 0.17
@aschaffer
Merge pull request #38 from rapidsai/branch-0.18
a584e0b 
Merge latest branch-0.18
@aschaffer
Merge pull request #39 from rapidsai/branch-0.18
338b2d4 
Update forked branch-0.18
@aschaffer
Merge pull request #40 from rapidsai/branch-0.18
efd7e9a 
Update forked branch-0.18 from release
@aschaffer
Merge pull request #41 from rapidsai/branch-0.19
a23ce0d 
Update branch-0.19 from release
@aschaffer
Merge pull request #42 from rapidsai/branch-0.19
22b32d8 
update forked from release branch-0.19
@aschaffer
Merge pull request #43 from rapidsai/branch-0.19
e5042fc 
Merge from release branch-0.19
@aschaffer
Merge pull request #44 from rapidsai/branch-0.19
ca02ee4 
Merge latest rapidsai:branch-0.19 into aschaffer:branch-0.19
@aschaffer
Merge pull request #45 from rapidsai/branch-0.20
79409e7 
Create forked branch-0.20
@aschaffer
Merge pull request #46 from rapidsai/branch-0.20
360c293 
merge latest release branch-0.20
@aschaffer
Merge pull request #47 from rapidsai/branch-0.20
e57f261 
Merge latest rapidsai/branch-0.20 into forked branch-0.20
@aschaffer
Merge pull request #48 from rapidsai/branch-21.08
a63cdea 
aschaffer/21.08 <- rapidsai/21.08
@aschaffer
Merge branch 'rapidsai:branch-21.08' into branch-21.08
7e6bf4c
@aschaffer
Merge branch 'rapidsai:branch-21.08' into branch-21.08
6f3b342
@aschaffer
Merge branch 'rapidsai:branch-21.10' into branch-21.10
fbf5f36
@aschaffer
Merge branch 'rapidsai:branch-21.10' into branch-21.10
ffacad0
@aschaffer
Merge branch 'rapidsai:branch-21.10' into branch-21.10
8db2e74
@aschaffer
Merge branch 'rapidsai:branch-21.12' into branch-21.12
e560fd8
@aschaffer
Merge branch 'rapidsai:branch-22.02' into branch-22.02
a897416
@aschaffer
Merge branch 'rapidsai:branch-22.02' into fea_ext_mnmg_mfg
6a8000c
@aschaffer
Merge branch 'rapidsai:branch-22.04' into fea_ext_mnmg_mfg
8b363bb
@aschaffer
Added NBR Sampling header and API.
a8b3f84
@aschaffer
update on uniform_nbr_sample() to reflect latest requirements.
1a7660c
@aschaffer
project() version with zip iterators.
72a90d0
@aschaffer
Added degree stub calls.
a76a743
@aschaffer
Merge branch 'branch-22.04' of github.com:rapidsai/cugraph into fea_e…
1ad5e71 
…xt_mnmg_mfg
@aschaffer
Added steps up-to gather.
fd0a383
@aschaffer
Union step.
7434d33
@aschaffer
Minor fixes and updates.
3a0953b
@aschaffer
Added _impl/API. And shuffle scaffolding.
acd2b52
seunghwak
seunghwak reviewed Mar 15, 2022
View reviewed changes
cpp/include/cugraph/algorithms.hpp Show resolved Hide resolved
cpp/include/cugraph/algorithms.hpp Outdated Show resolved Hide resolved
cpp/include/cugraph/algorithms.hpp Outdated Show resolved Hide resolved
cpp/src/sampling/nbr_sampling.cu Outdated Show resolved Hide resolved
cpp/src/sampling/nbr_sampling.cuh Outdated Show resolved Hide resolved
@aschaffer
Copy link
Collaborator Author

rerun tests

1 similar comment
@BradReesWork
Copy link
Member

rerun tests

@aschaffer
Copy link
Collaborator Author

Note: Requires #2110 merged to resolve current CI failures.

@aschaffer aschaffer requested a review from a team as a code owner March 16, 2022 14:37
@aschaffer
Copy link
Collaborator Author

rerun tests

1 similar comment
@aschaffer
Copy link
Collaborator Author

rerun tests

rlratzel
rlratzel approved these changes Mar 16, 2022
View reviewed changes
Copy link
Contributor

@rlratzel rlratzel left a comment

Choose a reason for hiding this comment

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

My only feedback was wrt the rank naming discussion, but approving so it gets merged.

@aschaffer aschaffer mentioned this pull request Mar 17, 2022
Closed
@BradReesWork
Copy link
Member

@gpucibot merge

@jjacobelli
Copy link
Contributor

Removing ops-codeowners from the required reviews since it doesn't seem there are any file changes that we're responsible for.

@jjacobelli jjacobelli removed the request for review from a team March 17, 2022 09:26
@jjacobelli
Copy link
Contributor

@gpucibot merge

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

@seunghwak seunghwak seunghwak approved these changes

@kaatish kaatish kaatish approved these changes

@rlratzel rlratzel rlratzel approved these changes

@ChuckHastings ChuckHastings ChuckHastings approved these changes

Assignees

@aschaffer aschaffer

Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Milestone
22.04
Development

Successfully merging this pull request may close these issues.
8 participants
@aschaffer @codecov-commenter @BradReesWork @jjacobelli @kaatish @rlratzel @ChuckHastings @seunghwak