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.

Sign up for GitHub

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

Jump to bottom

[ENH] MNMG Core Number #2169

Closed
2 of 4 tasks
BradReesWork opened this issue Mar 29, 2022 · 3 comments
Closed
2 of 4 tasks

[ENH] MNMG Core Number #2169

BradReesWork opened this issue Mar 29, 2022 · 3 comments
Labels
feature request New feature or request
Milestone
22.10

Comments

@BradReesWork
Copy link
Member

BradReesWork commented Mar 29, 2022 •
edited
Loading

Enhance Core Number to be MNMG
Code across all libraries

  • libcugraph
  • libcugraph_c
  • pylibcugraph
  • cugraph
@BradReesWork BradReesWork added the feature request New feature or request label Mar 29, 2022
@BradReesWork BradReesWork added this to the 22.08 milestone Mar 29, 2022
@seunghwak
Copy link
Contributor

libcugraph part is complete (https://github.com/rapidsai/cugraph/blob/branch-22.04/cpp/include/cugraph/algorithms.hpp#L1533).

@github-actions
Copy link

This issue has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d if there is no activity in the next 60 days.

@betochimas betochimas mentioned this issue Jun 3, 2022
Merged
rapids-bot bot pushed a commit that referenced this issue Jul 18, 2022
@betochimas
Add core number to the C API (#2338)
23f9444 
This addresses parts of #2169, notably the libcugraph_c API for core number, implemented in `src/c_api/core_number.cpp`, with relevant header updates. Testing had been started for the sg variant in `tests/c_api/core_number_test.c`, though results are currently not matching up with what's expected. Testing for the mg variant in `tests/c_api/mg_core_number_test.c` has not been started.

Also, the pylibcugraph wrapper for core number is included (not tested yet), so this PR can include both APIs or be split into 2 separate PRs, one for each level.

Authors:
  - Dylan Chima-Sanchez (https://github.com/betochimas)
  - Chuck Hastings (https://github.com/ChuckHastings)

Approvers:
  - Seunghwa Kang (https://github.com/seunghwak)
  - Joseph Nke (https://github.com/jnke2016)
  - Chuck Hastings (https://github.com/ChuckHastings)
  - Rick Ratzel (https://github.com/rlratzel)

URL: #2338
@jarmak-nv jarmak-nv modified the milestones: 22.08, 22.10 Aug 8, 2022
@rlratzel
Copy link
Contributor

This is closed now that #2414 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request New feature or request
Projects
None yet
Milestone
22.10
Development

No branches or pull requests
4 participants
@rlratzel @BradReesWork @seunghwak @jarmak-nv