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

Merging 6 commits from Aug 19-22 #106

Merged
merged 6 commits into from
Aug 22, 2023
Merged

Merging 6 commits from Aug 19-22 #106

merged 6 commits into from
Aug 22, 2023

Conversation

betochimas
Copy link
Owner

No description provided.

naimnv and others added 6 commits August 19, 2023 16:34
…i#3811)

cugraph-dgl is not supported for CUDA 12 yet and but the doc build use rapidsai/ci:latest which is on CUDA 12 now.

Authors:
  - Naim (https://github.com/naimnv)

Approvers:
  - AJ Schmidt (https://github.com/ajschmidt8)
  - Brad Rees (https://github.com/BradReesWork)

URL: rapidsai#3811
Closes rapidsai#2811 

This PR adds `aiohttp` and `requests` as hard dependencies to cugraph. Without these packages, `cugraph.datasets` is unable to download missing datasets. `cugraph.datasets` uses `cudf.read_csv` to download files from s3.

Authors:
  - ralph (https://github.com/nv-rliu)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - Rick Ratzel (https://github.com/rlratzel)
  - Brad Rees (https://github.com/BradReesWork)
  - Ray Douglass (https://github.com/raydouglass)

URL: rapidsai#3781
…apidsai#3799)

closes rapidsai#3798 

This PR adds missing copyright and license text to \_\_init\_\_.py package files.

The starting copyright year is based on the earliest entry in the "git log \_\_init\_\_.py" output.

Authors:
  - Rick Ratzel (https://github.com/rlratzel)

Approvers:
  - Brad Rees (https://github.com/BradReesWork)

URL: rapidsai#3799
closes rapidsai#2416

Removes a dependency when running python tests, which will speed up test runs. Also introduces a new testing component (`Resultset`) to clean up unit tests by generating golden results and storing them locally/on the cloud. This PR is focused on tests from the `traversal` category.

NOTE: Devs will download specific golden results from the cloud to avoid local overhead.

Authors:
  - Dylan Chima-Sanchez (https://github.com/betochimas)

Approvers:
  - Rick Ratzel (https://github.com/rlratzel)

URL: rapidsai#3705
…3.2 (rapidsai#3808)

This is one step to support rapidsai#3770 

Note that NetworkX version 3.2 is currently under development.

CC @rlratzel

Authors:
  - Erik Welch (https://github.com/eriknw)

Approvers:
  - Rick Ratzel (https://github.com/rlratzel)

URL: rapidsai#3808
This PR fixes: rapidsai#3784 and speeds up MFG creation by `3.5x` . 

Todo:
- [x] Add tests


Benchmarked on 6_462_743_488 edges with a batch size of `128` on a 1 V100:

Before PR Times:
```
1min 17s
```

After PR Times:
```
22 s
```

See link: https://gist.github.com/VibhuJawa/4852203f2e96de09d84d698af945682d


**Profiling:**

After PR: rapidsai#3810

<img width="1252" alt="image" src="https://github.com/rapidsai/cugraph/assets/4837571/4cbe5153-4251-4195-9471-c60d11cdf7e9">


<img width="1252" alt="image" src="https://github.com/rapidsai/cugraph/assets/4837571/ad019f47-6ccf-45b2-b866-9a2f4f16bc9b">


Profile of splitting df into tensors : 

<img width="781" alt="image" src="https://github.com/rapidsai/cugraph/assets/4837571/82c401d6-1fca-44da-871f-ebb163a464ba">

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

Approvers:
  - Alex Barghi (https://github.com/alexbarghi-nv)
  - Tingyu Wang (https://github.com/tingyu66)

URL: rapidsai#3810
@betochimas betochimas merged commit c5cb00e into cu-nx-notebook Aug 22, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants