Skip to content

[REVIEW] update subgraph tests and remove legacy pagerank#1378

Merged
rapids-bot[bot] merged 4 commits intorapidsai:branch-0.18from
Iroy30:update_subgraph_remove_pagerank
Feb 5, 2021
Merged

[REVIEW] update subgraph tests and remove legacy pagerank#1378
rapids-bot[bot] merged 4 commits intorapidsai:branch-0.18from
Iroy30:update_subgraph_remove_pagerank

Conversation

@Iroy30
Copy link
Copy Markdown
Contributor

@Iroy30 Iroy30 commented Feb 3, 2021

No description provided.

@Iroy30 Iroy30 requested a review from a team as a code owner February 3, 2021 18:55
@BradReesWork BradReesWork added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Feb 4, 2021
@BradReesWork BradReesWork added this to the 0.18 milestone Feb 4, 2021
@BradReesWork BradReesWork requested review from aschaffer, jnke2016 and rlratzel and removed request for jnke2016 February 4, 2021 21:44
@Iroy30 Iroy30 requested review from a team as code owners February 4, 2021 22:55
@Iroy30 Iroy30 changed the title [WIP] update subgraph tests and remove legacy pagerank [REVIEW] update subgraph tests and remove legacy pagerank Feb 4, 2021
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1378 (3fccf7b) into branch-0.18 (2fb0725) will increase coverage by 0.06%.
The diff coverage is 56.79%.

Impacted file tree graph

@@               Coverage Diff               @@
##           branch-0.18    #1378      +/-   ##
===============================================
+ Coverage        60.38%   60.44%   +0.06%     
===============================================
  Files               67       69       +2     
  Lines             3029     3120      +91     
===============================================
+ Hits              1829     1886      +57     
- Misses            1200     1234      +34     
Impacted Files Coverage Δ
python/cugraph/centrality/__init__.py 100.00% <ø> (ø)
python/cugraph/dask/structure/renumber.py 0.00% <0.00%> (ø)
python/cugraph/link_analysis/pagerank.py 100.00% <ø> (ø)
python/cugraph/comms/comms.py 34.52% <25.00%> (ø)
python/cugraph/dask/common/input_utils.py 23.07% <28.57%> (+1.14%) ⬆️
python/cugraph/utilities/utils.py 66.92% <30.76%> (-4.63%) ⬇️
python/cugraph/dask/common/mg_utils.py 37.50% <38.09%> (-2.50%) ⬇️
python/cugraph/community/spectral_clustering.py 72.54% <38.46%> (-11.67%) ⬇️
python/cugraph/structure/number_map.py 59.20% <50.00%> (+3.24%) ⬆️
python/cugraph/structure/graph.py 66.99% <76.47%> (+0.19%) ⬆️
... and 14 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 95d4dec...3fccf7b. Read the comment docs.

@BradReesWork BradReesWork requested a review from afender February 5, 2021 14:50
Copy link
Copy Markdown
Member

@afender afender left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@BradReesWork
Copy link
Copy Markdown
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit d8d35e4 into rapidsai:branch-0.18 Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants