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

Update GNN stellargraph with DGL #1032

Merged
merged 116 commits into from
Aug 29, 2023
Merged

Update GNN stellargraph with DGL #1032

merged 116 commits into from
Aug 29, 2023

Conversation

tzemicheal
Copy link
Contributor

@tzemicheal tzemicheal commented Jul 11, 2023 •
edited by dagardner-nv
Loading

Description

This PR replaces stellargraph reference in the training & validation scripts. This will address the issue #945 & #925

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@tzemicheal tzemicheal added feature request New feature or request improvement Improvement to existing functionality 2 - In Progress labels Jul 11, 2023
@tzemicheal tzemicheal requested a review from a team as a code owner July 11, 2023 01:20
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@tzemicheal tzemicheal added non-breaking Non-breaking change and removed feature request New feature or request labels Jul 11, 2023
@mdemoret-nv
Copy link
Contributor

FYI, we will be targeting 23.11 for this PR

@mdemoret-nv mdemoret-nv changed the base branch from branch-23.07 to branch-23.11 July 14, 2023 18:59
@tzemicheal tzemicheal requested review from a team as code owners July 19, 2023 21:12
@tzemicheal tzemicheal added breaking Breaking change and removed breaking Breaking change labels Jul 19, 2023
@tzemicheal tzemicheal requested a review from gbatmaz July 24, 2023 15:05
tzemicheal and others added 16 commits August 28, 2023 13:43
@tzemicheal
Merge pull request #9 from dagardner-nv/david-fsi_dgl-patch8
ac05828 
Consolidate common sections of HinSAGE & HeteroRGCN into a new base class
@dagardner-nv
More type hints
fea1f1d
@dagardner-nv
Document manual_seed fixture
3965f43
@dagardner-nv
Merge branch 'fsi_dgl' of github.com:tzemicheal/Morpheus into david-f…
f5c5149 
…si_dgl-patch9
@dagardner-nv
Enforce dir for model_dir arg
fbc86a9
@dagardner-nv
Add docstring for remove_module
e38d24a
@tzemicheal
Merge pull request #10 from dagardner-nv/david-fsi_dgl-patch9
bb20434 
More type hints
@tzemicheal
workspace update
7567042
@tzemicheal
Merge branch 'fsi_dgl' of https://github.com/tzemicheal/Morpheus into…
e64ee7c 
… fsi_dgl
@tzemicheal
Track lfs val/train dataset
d91c860
@tzemicheal
modified: examples/gnn_fraud_detection_pipeline/run.py
4118b70
@tzemicheal
Merge branch 'fsi_dgl' of github.com:tzemicheal/Morpheus into fsi_dgl
62f73aa
@dagardner-nv
Merge branch 'branch-23.11' into david-fsi-dgl-lfs
42b68ff
@dagardner-nv
Revert "Track lfs val/train dataset"
3798562 
This reverts commit d91c860.
@dagardner-nv
Merge branch 'david-fsi-dgl-lfs' of github.com:dagardner-nv/Morpheus …
f3a5acd 
…into david-fsi-dgl-lfs
@tzemicheal
Merge pull request #11 from dagardner-nv/david-fsi-dgl-lfs
525b0c6 
Revert moving csv files to LFS
@dagardner-nv
Copy link
Contributor

/OK to test

@copy-pr-bot
Copy link

copy-pr-bot bot commented Aug 29, 2023

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@dagardner-nv
Copy link
Contributor

/ok to test

@dagardner-nv
Copy link
Contributor

/ok to test

@dagardner-nv
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit c89d4c1 into nv-morpheus:branch-23.11 Aug 29, 2023
6 checks passed
This was referenced Dec 7, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdemoret-nv mdemoret-nv mdemoret-nv approved these changes

@gbatmaz gbatmaz gbatmaz approved these changes

@dagardner-nv dagardner-nv dagardner-nv approved these changes

Assignees

@tzemicheal tzemicheal

Labels
improvement Improvement to existing functionality non-breaking Non-breaking change
Projects
Morpheus Boards
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tzemicheal @mdemoret-nv @dagardner-nv @gbatmaz