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

[RELEASE] wholegraph v24.06 #179

Merged
merged 20 commits into from
Jun 5, 2024
Merged

[RELEASE] wholegraph v24.06 #179

merged 20 commits into from
Jun 5, 2024

Conversation

raydouglass
Copy link
Member

❄️ Code freeze for branch-24.06 and v24.06 release

What does this mean?

Only critical/hotfix level issues should be merged into branch-24.06 until release (merging of this PR).

What is the purpose of this PR?

  • Update documentation
  • Allow testing for the new release
  • Enable a means to merge branch-24.06 into main for the release

raydouglass and others added 20 commits March 15, 2024 12:05
Forward-merge branch-24.04 to branch-24.06
Forward-merge branch-24.04 to branch-24.06
Forward-merge branch-24.04 into branch-24.06
Forward-merge branch-24.04 into branch-24.06
Forward-merge branch-24.04 into branch-24.06
mkl2024.1.0 conflicts with pytorch, this PR constrains mkl<2024.1.0 in the test scripts where pytorch is required.

Authors:
  - https://github.com/linhu-nv

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

URL: #162
mkl2024.1.0 conflicts with pytorch, this PR constrains mkl<2024.1.0 in the test scripts where pytorch is required.

Authors:
  - https://github.com/linhu-nv

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

URL: #162
fix to issue 148[https://github.com/rapidsai/wholegraph/issues/148](url), remove unnecessary sync between thrust ops and host cpu threads

Authors:
  - https://github.com/linhu-nv

Approvers:
  - Chuang Zhu (https://github.com/chuangz0)

URL: #160
fix to[ issue 76](#76), which allows temp_memory_handler to allocate memory for multiple times.

Authors:
  - https://github.com/linhu-nv

Approvers:
  - Chuang Zhu (https://github.com/chuangz0)

URL: #161
The `sysroot*` syntax is getting phased out (conda-forge/conda-forge.github.io#2102).
The recommendation is to move to `{{ stdlib("c") }}`.

Ref rapidsai/build-planning#39

Authors:
  - Philip Hyunsu Cho (https://github.com/hcho3)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - https://github.com/jakirkham
  - Ray Douglass (https://github.com/raydouglass)

URL: #164
This script appears to have never been used, and with the usage of `pre-commit-hooks`' `verify-copyright` hook is now obsolete.

Authors:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

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

URL: #149
…n_size (#172)

quick fix to a map_indice bug && add comment for parameter round_robin_size

Authors:
  - https://github.com/linhu-nv

Approvers:
  - Chuang Zhu (https://github.com/chuangz0)

URL: #172
A quick fixes to this issue (#168). Set correct default wholememory tensor gather results data type.

Authors:
  - https://github.com/linhu-nv

Approvers:
  - Chuang Zhu (https://github.com/chuangz0)

URL: #173
In continuous/chunked host mode, sorting indices before gathering can improve bandwidth by enhancing memory locality.

Authors:
  - https://github.com/zhuofan1123

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

URL: #174
This PR introduces support for distributed graph sampling (via NCCL backend). The initial implementation focuses on the uniform neighbor sampler.
We are going to extend it to support other samplers in future.

Highlights:
- Distributed Graph Storage: Now, the graph structure (represented by `row_ptr` and `col_indx` tensors) can be stored as wholememory arrays in a distributed fashion with even distribution across ranks (support both `cpu` and `cuda` storage type).
- Distributed Sampling: The sampling process leverages the existing wholegraph gather function to collect the sampled nodes and edges across all ranks.
- Uniform Neighbor Sampler Support: Currently, only the uniform neighbor sampler is supported.

cc. @linhu-nv @dongxuy04 @BradReesWork  @nvcastet @TristonC

Authors:
  - Chang Liu (https://github.com/chang-l)

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

URL: #171
@raydouglass raydouglass requested a review from a team as a code owner May 31, 2024 13:19
Copy link

copy-pr-bot bot commented May 31, 2024

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.

@raydouglass raydouglass merged commit e536910 into main Jun 5, 2024
463 of 466 checks passed
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 this pull request may close these issues.

9 participants