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

use "random" from RAFT API #7

Closed
Tracked by #48
teju85 opened this issue Aug 16, 2022 · 1 comment
Closed
Tracked by #48

use "random" from RAFT API #7

teju85 opened this issue Aug 16, 2022 · 1 comment
Assignees
@dongxuy04
Labels
good first issue Good for newcomers tech-debt Cleanup tasks

Comments

@teju85
Copy link
Member

teju85 commented Aug 16, 2022

WM has its own RNG functions inside wholegraph/random.cuh. This is unnecessary IMO and we should instead be using RAFT's random namespace
@teju85 teju85 added good first issue Good for newcomers tech-debt Cleanup tasks labels Aug 16, 2022
@BradReesWork BradReesWork mentioned this issue Jul 10, 2023
Closed
@BradReesWork BradReesWork mentioned this issue Aug 3, 2023
Open
linhu-nv added a commit to linhu-nv/wholegraph that referenced this issue Aug 11, 2023
@linhu-nv
replace RNG with raft RNG generator, issue rapidsai#7 and rapidsai#23 …
b8f20d5 
…for wholegraph 23.10
This was referenced Oct 9, 2023
Closed
Merged
rapids-bot bot pushed a commit that referenced this issue Nov 21, 2023
@linhu-nv
Use RNG (random number generator) provided by RAFT (#79)
9dc74b7 
Replace the random number generator (RNG) implemented by wholegraph with RNG provided by RAFT. 
It is put forward by issue #7 and issue #23.

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

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

URL: #79
@BradReesWork
Copy link
Member

closed by #79

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@dongxuy04 dongxuy04

Labels
good first issue Good for newcomers tech-debt Cleanup tasks
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@teju85 @BradReesWork @dongxuy04