[ENH] pylibcugraph should run doctests as part of its test suite #2193
Assignees
Labels
? - Needs Triage
Need team to review and classify
improvement
Improvement / enhancement to an existing function
non-breaking
Non-breaking change
Milestone
Comments
rlratzel
added
? - Needs Triage
rapids-bot bot
pushed a commit
that referenced
this issue
Apr 30, 2022
This PR incorporates suggested changes from issue #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 #2227. This PR also closes #2193. Authors: - https://github.com/betochimas Approvers: - Rick Ratzel (https://github.com/rlratzel) URL: #2194
This was referenced May 2, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Like #2014 did for
cugraph,pylibcugraphshould also have doctest runs as part of its test suite.The text was updated successfully, but these errors were encountered: