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

Rename primitive functions + Move pylibcugraph APIs out of experimental + Python code cleanup #57

Merged
merged 3 commits into from
May 2, 2022

Conversation

betochimas
Copy link
Owner

No description provided.

seunghwak and others added 3 commits April 28, 2022 01:43
Partially address rapidsai#2003
Rename
- copy_v_transform_reduce_in|out_nbr => transform_reduce_incoming|outgoing_e_of_v
- copy_v_transform_reduce_key_aggregated_out_nbr => transform_reduce_key_aggregated_outgoing_e_of_v
- update_frontier_v_push_if_out_nbr => update_frontier_from_outgoing_e_of_v

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

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

URL: rapidsai#2234
…namespace (rapidsai#2232)

This PR addresses and closes rapidsai#2202. Specifically, this PR does the following:
- Adds current experimental pylibcugraph APIs outside of experimental namespace, without removing the ones inside that namespace (duplicating in a sense)
- The ones promoted outside have no warning wrappers attached
- The original APIs have a different warning applied to them, stating that their version will be removed within a few releases, leaving the promoted ones intact

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

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

URL: rapidsai#2232
This PR incorporates suggested changes from issue rapidsai#2037, opened back in 22.02, as well as cleaning up documentation and testing code from 22.04 work.

Specifically, this PR does the following:
- Removes worker helpers from MNMG algos in the online documentation (e.g. `call_katz_centrality`, `call_louvain`, `call_wcc`)
- Adds helpers from comms such as `get_comms`, `get_session_id`, `get_2D_partition`, is_initialized that were previously missing
- Adds helpers from NumberMap such as `from_internal_vertex_id`, `to_internal_vertex_id`, `renumber`, `set_renumbered_col_names`
- Add MNMG Hits and Node2vec to docs
- Added docstring descriptions and examples to methods that were missing them
- Fixed docstring examples in `egonet.py`, `node2vec.py`
- Adds `pylibcugraph` docstring examples to the list of examples tested in `cugraph/tests/test_doctests.py`
- Adds support for MG Dask docstring example testing, tested in newly created `cugraph/tests/dask/test_mg_doctests.py` 

This PR addresses the items from and closes rapidsai#2227.
This PR also closes rapidsai#2193.

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

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

URL: rapidsai#2194
@betochimas betochimas merged commit 9446aff into branch-22.06-mst May 2, 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