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

Document adding cuML and fix the gnn_fraud_detection_pipeline example #758

Merged
merged 25 commits into from
Mar 29, 2023
Merged
Show file tree
Hide file tree
Changes from 16 commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
04313be
Add cuml to the default env
dagardner-nv Mar 13, 2023
e97df4a
Add version restrictions per https://docs.rapids.ai/notices/rsn0026/
dagardner-nv Mar 13, 2023
9f471fd
Add cuml to conda recipe as suggested by @jimux
dagardner-nv Mar 13, 2023
f8a71c1
Merge branch 'branch-23.03' into david-cuml-dep-741
dagardner-nv Mar 13, 2023
ba76a99
Merge branch 'branch-23.03' into david-cuml-dep-741 [no ci]
dagardner-nv Mar 24, 2023
050d744
Revert "Add cuml to conda recipe as suggested by @jimux" [no ci]
dagardner-nv Mar 24, 2023
4463cd9
Set cuml dep to match our version fo rapids [no ci]
dagardner-nv Mar 24, 2023
8256465
Remove cuml dep [no ci]
dagardner-nv Mar 24, 2023
882f209
Merge branch 'branch-23.03' into david-cuml-dep-741
dagardner-nv Mar 24, 2023
7f9e667
Merge branch 'branch-23.03' into david-cuml-dep-741
dagardner-nv Mar 27, 2023
904d94d
wip [no ci]
dagardner-nv Mar 27, 2023
5948627
Bump protobuf version to match that needed by tesnorflow 2.12
dagardner-nv Mar 28, 2023
9663c9b
Formatting
dagardner-nv Mar 28, 2023
50fb63d
Pull in stellargraph and tensorflow from pypi, the versions on conda-…
dagardner-nv Mar 28, 2023
04745f6
Work-around for apparent change in tensorflow
dagardner-nv Mar 28, 2023
c16a913
Merge branch 'branch-23.03' into david-cuml-dep-741
dagardner-nv Mar 28, 2023
d04c8d0
wip
dagardner-nv Mar 28, 2023
2d8d421
Merge branch 'david-cuml-dep-741' of github.com:dagardner-nv/Morpheus…
dagardner-nv Mar 28, 2023
07beac2
wip [no ci]
dagardner-nv Mar 28, 2023
c17e44b
Merge branch 'branch-23.03' into david-cuml-dep-741 [no ci]
dagardner-nv Mar 28, 2023
08a467b
Add comments explaining the versions [no ci]
dagardner-nv Mar 28, 2023
56a10f9
Add comments explaining the versions
dagardner-nv Mar 28, 2023
55a1b5f
Merge branch 'david-cuml-dep-741' of github.com:dagardner-nv/Morpheus…
dagardner-nv Mar 28, 2023
b26016f
document installing cuml
dagardner-nv Mar 28, 2023
e49e56d
Merge branch 'branch-23.03' into david-cuml-dep-741
dagardner-nv Mar 28, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion docker/conda/environments/cuda11.8_dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,10 @@ dependencies:
- gxx_linux-64=11.2
- include-what-you-use=0.18
- isort
# If cuml is installed, we need to constrain these two libraries. See:
# https://docs.rapids.ai/notices/rsn0026/ for libcusolver & libcusparse
- libcusolver<=11.4.1.48
- libcusparse<12
- librdkafka=1.7.0
- mlflow>1.29,<2
- mrc=23.03
Expand All @@ -71,7 +75,7 @@ dependencies:
- pip
- pkg-config # for mrc cmake
- pluggy=1.0
- protobuf=3.20.2
- protobuf=3.20.3
- pybind11-stubgen=0.10.5
- pydot
- pytest
Expand Down
2 changes: 1 addition & 1 deletion docker/conda/environments/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jupyter-core>=4.11.2,<5.0
jupyterlab
nvidia-pyindex
# Duplicated in conda dev to ensure parity with libprotobuf
protobuf==3.20.2
protobuf==3.20.3
pytest-benchmark>=4.0
torch==1.13.1+cu116
tritonclient[all]==2.17.*
Expand Down
7 changes: 6 additions & 1 deletion examples/gnn_fraud_detection_pipeline/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,10 @@ export MORPHEUS_ROOT=$(pwd)
Use Morpheus to run the GNN fraud detection Pipeline with the transaction data. A pipeline has been configured in `run.py` with several command line options:

```bash
cd ${MORPHEUS_ROOT}/examples/gnn_fraud_detection_pipeline
python run.py --help
```
```
Usage: run.py [OPTIONS]

Options:
Expand All @@ -61,8 +64,10 @@ Options:
To launch the configured Morpheus pipeline with the sample data that is provided at `$MORPHEUS_ROOT/models/dataset`, run the following:

```bash

cd ${MORPHEUS_ROOT}/examples/gnn_fraud_detection_pipeline
python run.py
```
```
====Building Pipeline====
Added source: <from-file-0; FileSourceStage(filename=validation.csv, iterative=None, file_type=auto, repeat=1, filter_null=False)>
└─> morpheus.MessageMeta
Expand Down
15 changes: 10 additions & 5 deletions examples/gnn_fraud_detection_pipeline/requirements.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,16 @@
channels:
- rapidsai
- nvidia
- nvidia/label/dev # For pre-releases of MRC. Should still default to full releases if available
- stellargraph
- conda-forge
dependencies:
- chardet=5.0.0
- cuml=22.08
- stellargraph=1.2.1
- tensorflow=2.9.1
- cuml=22.10
# https://docs.rapids.ai/notices/rsn0026/ for libcusolver & libcusparse
- libcusolver<=11.4.1.48
- libcusparse<12
- cuda-python<=11.7.0 # Remove when Issue #251 is closed
- pip:
- stellargraph==1.2.1
- tensorflow==2.12.0
# Duplicated in conda dev to ensure parity with libprotobuf
- protobuf==3.20.3
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
import typing

import mrc
import numpy as np

import cudf

Expand Down Expand Up @@ -95,7 +96,7 @@ def _inductive_step_hinsage(
generator = HinSAGENodeGenerator(graph, self._batch_size, self._sample_size, head_node_type=self._target_node)
test_gen_not_shuffled = generator.flow(node_identifiers, shuffle=False)

inductive_emb = trained_model.predict(test_gen_not_shuffled)
inductive_emb = np.concatenate([trained_model.predict(row[0]) for row in test_gen_not_shuffled])
inductive_emb = cudf.DataFrame(inductive_emb, index=node_identifiers)

return inductive_emb
Expand Down