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 May 10-13 commits #68

Merged
merged 8 commits into from
May 16, 2022
Merged

Conversation

betochimas
Copy link
Owner

No description provided.

betochimas and others added 8 commits May 10, 2022 17:50
This PR adds Degree Centrality to the cugraph software stack, and closes rapidsai#2145 when merged.

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

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

URL: rapidsai#2259
…2257)

Resolve inconsistencies in reduction support in primitives (partially addresses rapidsai#2003).
  * If `reduce_op` is missing, + is assumed.
  * If `init` is missing, T{} is assumed.
  * Provides a pre-defined set of reduction operations in `reduce_op.cuh`
  * Provides a guidance on writing custom reduction operators.
  * Fix a bug in reduction support for thrust::tuple (std::numeric_limits does not work properly for thrust::tuple, std::numeric_limits should be applied to individual elements of thrust::tuple).

Authors:
  - Seunghwa Kang (https://github.com/seunghwak)

Approvers:
  - Chuck Hastings (https://github.com/ChuckHastings)
  - Kumar Aatish (https://github.com/kaatish)

URL: rapidsai#2257
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: rapidsai#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 rapidsai#2240
closes rapidsai#2174 
closes rapidsai#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: rapidsai#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: rapidsai#2230
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: rapidsai#2258
@betochimas betochimas merged commit d9e2301 into branch-22.06-eigen May 16, 2022
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