Skip to content
This repository has been archived by the owner on Nov 25, 2024. It is now read-only.

Replace all the GNN layer ops kernels with the corresponding APIs from cugraph-ops #17

Closed
Tracked by #38
teju85 opened this issue May 12, 2023 · 1 comment · Fixed by #24
Closed
Tracked by #38

Replace all the GNN layer ops kernels with the corresponding APIs from cugraph-ops #17

teju85 opened this issue May 12, 2023 · 1 comment · Fixed by #24
Assignees

Comments

@teju85
Copy link
Member

teju85 commented May 12, 2023

All (if not most) of the GNN layer ops in cugraph-ops should already help do the job, for which we have specific kernels in WholeGraph. Thus, there's lot of duplication of code between these 2 projects. We must further refactor WG GNN layer ops in order to use the ones from cugraph-ops instead of having to maintain specific kernels for these in WG.

  1. If not attempted already, please prepare a plan to replace these kernels with the corresponding calls from cugraph-ops in the next release cycle?
  2. If attempted, can you file issues against cugraphops for any missing features that are preventing this from happening?
@dongxuy04
Copy link
Contributor

@teju85 Have replaced GNN layer related ops with cugraph-ops from python level. Could you help check that and maybe we can close this issue? Thanks!

@rapids-bot rapids-bot bot closed this as completed in #24 Jul 28, 2023
rapids-bot bot pushed a commit that referenced this issue Jul 28, 2023
This branch is a collection of all the refactoring improvements.  

closes #13 
closes #14 
closes #17 
closes #18 
closes rapidsai/cugraph-ops#492

Authors:
  - Brad Rees (https://github.com/BradReesWork)
  - https://github.com/dongxuy04
  - Junhong Liu (https://github.com/hello-11)
  - Tingyu Wang (https://github.com/tingyu66)

Approvers:
  - Tingyu Wang (https://github.com/tingyu66)
  - https://github.com/dongxuy04
  - AJ Schmidt (https://github.com/ajschmidt8)

URL: #24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants