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

[FEA] Reformat graph_store and move it into DGL extensions #2838

Closed
4 of 6 tasks
Tracked by #3255
VibhuJawa opened this issue Oct 21, 2022 · 1 comment
Closed
4 of 6 tasks
Tracked by #3255

[FEA] Reformat graph_store and move it into DGL extensions #2838

VibhuJawa opened this issue Oct 21, 2022 · 1 comment
Assignees
@VibhuJawa
Labels
Story Tracking issues as a story level - likely to exist within a single release.

Comments

@VibhuJawa
Copy link
Member

VibhuJawa commented Oct 21, 2022 •
edited
Loading

Describe the solution you'd like and any additional context

For Developers below this line

  • [] Code passes CI
  • [] Code uses Graph Primitives
  • [] Code in C++/CUDA layer
  • [] Code in C layer
  • [] Code in pylibcugraph Python layer
  • Code in cugraph Python layer
  • [] Documentation
  • [] Test cases at each layer (for MG tests, a note in the PR description indicating the new/existing MG tests were run and passed)
@VibhuJawa VibhuJawa added the ? - Needs Triage Need team to review and classify label Oct 21, 2022
@VibhuJawa VibhuJawa mentioned this issue Oct 21, 2022
Merged
@acostadon acostadon removed the ? - Needs Triage Need team to review and classify label Oct 21, 2022
rapids-bot bot pushed a commit that referenced this issue Oct 24, 2022
@VibhuJawa
Move 'graph_store.py' under dgl_extensions (#2839)
30afb48 
This PR resolves part of #2838 . 

 This helps with code cleanup and organization as we now support both PyG and DGL .

Authors:
  - Vibhu Jawa (https://github.com/VibhuJawa)

Approvers:
  - Alex Barghi (https://github.com/alexbarghi-nv)
  - Rick Ratzel (https://github.com/rlratzel)

URL: #2839
@VibhuJawa VibhuJawa self-assigned this Oct 27, 2022
@VibhuJawa VibhuJawa added this to the 22.12 milestone Oct 27, 2022
@VibhuJawa VibhuJawa modified the milestones: 22.12, 23.02 Nov 15, 2022
@VibhuJawa VibhuJawa modified the milestones: 23.02, 23.04 Dec 1, 2022
@kingmesal kingmesal mentioned this issue Feb 8, 2023
Closed
@VibhuJawa VibhuJawa modified the milestones: 23.04, 23.08 Apr 7, 2023
@rlratzel rlratzel added the Story Tracking issues as a story level - likely to exist within a single release. label Apr 7, 2023
@rlratzel rlratzel removed this from the 23.08 milestone Apr 7, 2023
@rlratzel
Copy link
Contributor

rlratzel commented Apr 7, 2023

@alexbarghi-nv mentioned the remaining work was done in 22.12.

@rlratzel rlratzel closed this as completed Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@VibhuJawa VibhuJawa

Labels
Story Tracking issues as a story level - likely to exist within a single release.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@rlratzel @VibhuJawa @kingmesal @acostadon