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

Python code cleanup across docs, wrappers, testing #2188

Closed
wants to merge 40 commits into from

Conversation

betochimas
Copy link
Contributor

@betochimas betochimas commented Apr 4, 2022

This PR aims to add suggested changes from issue #2037, opened back in 22.02 cycle, as well as adding new suggested changes from 22.04 and 22.06 work. Issue #2037 is expected to change alongside this PR, so it will be in draft mode for a while, and it's to be determined if this will fully close #2037

(OUTDATED: #2194 is the successor PR for this issue)

raydouglass and others added 30 commits August 26, 2020 12:04
[RELEASE] Re-release v0.15 [skip-ci]
[RELEASE] Build fix for v0.19 cugraph
@betochimas betochimas added doc Documentation non-breaking Non-breaking change labels Apr 5, 2022
@BradReesWork BradReesWork added this to the 22.06 milestone Apr 5, 2022
@codecov-commenter
Copy link

codecov-commenter commented Apr 5, 2022

Codecov Report

Merging #2188 (96e7fee) into branch-22.06 (017baab) will increase coverage by 0.35%.
The diff coverage is n/a.

❗ Current head 96e7fee differs from pull request most recent head 494853f. Consider uploading reports for the commit 494853f to get more accurate results

@@               Coverage Diff                @@
##           branch-22.06    #2188      +/-   ##
================================================
+ Coverage         72.41%   72.77%   +0.35%     
================================================
  Files               157      157              
  Lines             10496    10496              
================================================
+ Hits               7601     7638      +37     
+ Misses             2895     2858      -37     
Impacted Files Coverage Δ
...graph/cugraph/centrality/betweenness_centrality.py 89.65% <ø> (ø)
python/cugraph/cugraph/comms/comms.py 70.23% <ø> (+35.71%) ⬆️
python/cugraph/cugraph/community/egonet.py 97.36% <ø> (ø)
...ython/cugraph/cugraph/community/ktruss_subgraph.py 85.29% <ø> (ø)
...cugraph/cugraph/dask/centrality/katz_centrality.py 32.14% <ø> (+7.14%) ⬆️
python/cugraph/cugraph/linear_assignment/lap.py 71.42% <ø> (ø)
python/cugraph/cugraph/sampling/node2vec.py 91.11% <ø> (ø)
python/cugraph/cugraph/structure/hypergraph.py 21.53% <ø> (ø)
.../cugraph/tests/test_edge_betweenness_centrality.py 84.93% <0.00%> (+0.60%) ⬆️
...graph/cugraph/tests/test_betweenness_centrality.py 83.01% <0.00%> (+0.62%) ⬆️
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 017baab...494853f. Read the comment docs.

@betochimas betochimas closed this Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Documentation non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation and testing improvements
7 participants