pull latest updates #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This PR updates usage of RAFT RNG functionality throughout the cugraph code-base, including using the new RAFT RngState to pass state between cugraph and cugraph-ops. Finally, I removed unnecessary includes from `raft/random` where applicable. Authors: - Matt Joux (https://github.com/MatthiasKohl) Approvers: - Chuck Hastings (https://github.com/ChuckHastings) - Kumar Aatish (https://github.com/kaatish) URL: #2260
This PR allows our MG implementation to support undirected graph by symmetrizing the dataframe. This will be consistent with the SG implementation. It also - replaces the deprecated `cudf` call `append` by `concat` - update Neighborhood sampling's docstring and code to only support `int32` vertices and labels by Dylan closes #2240 closes #2174 closes #2264 by raising a `ValueError` and updating the docstring Authors: - Joseph Nke (https://github.com/jnke2016) - https://github.com/betochimas Approvers: - Rick Ratzel (https://github.com/rlratzel) URL: #2247
This PR convert the `from sources` build we are doing in GPU test job to a `conda build`. This is done for the following reasons: - This is required step to improve the Ops CI/CD setup to a more convenient pipeline - This is required to start using `conda` compilers and `mamba` to build RAPIDS packages - This prevent us from manually managing and installing the dependencies in GPU job - This ensure the packages can be installed - This ensure the tests are running and working against the package content and not the build results. Currently the Python packages are not tested. This may increase the global pipeline time, but the usage of `mamba` should resolve this as `mamba` is faster than `conda` to build packages Authors: - Jordan Jacobelli (https://github.com/Ethyling) Approvers: - AJ Schmidt (https://github.com/ajschmidt8) - Rick Ratzel (https://github.com/rlratzel) URL: #2230
Authors: - Seunghwa Kang (https://github.com/seunghwak) Approvers: - Chuck Hastings (https://github.com/ChuckHastings) URL: #2269
This PR will refactor the Uniform Neighborhood Sampling implementation to meet the new C API. Major elements: * Moved old implementation details into cugraph::detail::original * Edge ids will be passed in as the edge weight, to allow them to be controlled by the caller. Edge weight will be an integer type, but we will treat int32_t * as float * (or int64_t * as double *). The algorithms will be flagged so that they won't do computations on the weight if it is an edge id * Adding an SG implementation (only partially done as of the creation of this PR, will be finished before this PR is ready for review) Authors: - Chuck Hastings (https://github.com/ChuckHastings) Approvers: - Seunghwa Kang (https://github.com/seunghwak) URL: #2258
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.