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

Define C API for eigenvector centrality (#2180) #39

Merged
merged 1 commit into from
Apr 6, 2022

Conversation

betochimas
Copy link
Owner

This PR defines the C API for eigenvector centrality. It is the first of a series of PRs to address rapidsai#2146

It also does the following:

  • Renames cugraph_pagerank_result_t to cugraph_centrality_result_t to allow it to be reused for all centrality algorithms
  • Reorganizes the code for the results into a separate file
  • Provides an initial implementation of eigenvector centrality that can be used for testing the API. NOTE: It literally calls pagerank with some hardcoded parameters. The return formats for the algorithms are the same, so this will mimic the API behavior (although the scores will be different once eigenvector centrality is implemented
  • Provided a C unit test that validates the results (will need to be updated to validate eigenvector centrality results once the real algorithm is in place)

Note that this is a breaking change since it modifies the type of the pagerank result.

Authors:

Approvers:

URL: rapidsai#2180

This PR defines the C API for eigenvector centrality.  It is the first of a series of PRs to address rapidsai#2146

It also does the following:
* Renames `cugraph_pagerank_result_t` to `cugraph_centrality_result_t` to allow it to be reused for all centrality algorithms
* Reorganizes the code for the results into a separate file
* Provides an initial implementation of eigenvector centrality that can be used for testing the API.  NOTE: It literally calls pagerank with some hardcoded parameters.  The return formats for the algorithms are the same, so this will mimic the API behavior (although the scores will be different once eigenvector centrality is implemented
* Provided a C unit test that validates the results (will need to be updated to validate eigenvector centrality results once the real algorithm is in place)

Note that this is a breaking change since it modifies the type of the pagerank result.

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

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

URL: rapidsai#2180
@betochimas betochimas merged commit 66ba355 into branch-22.06-docs2 Apr 6, 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.

2 participants