cugraph icon indicating copy to clipboard operation
cugraph copied to clipboard

Refactor louvain

Open jnke2016 opened this issue 3 years ago • 2 comments

This PR refactors Louvain by leveraging the C API and does minor updates to the python tests closes #2493

jnke2016 avatar Sep 19 '22 04:09 jnke2016

Codecov Report

Base: 60.16% // Head: 59.94% // Decreases project coverage by -0.22% :warning:

Coverage data is based on head (7f9e91a) compared to base (3537690). Patch coverage: 53.91% of modified lines in pull request are covered.

:exclamation: Current head 7f9e91a differs from pull request most recent head ab6726f. Consider uploading reports for the commit ab6726f to get more accurate results

Additional details and impacted files
@@               Coverage Diff                @@
##           branch-22.10    #2705      +/-   ##
================================================
- Coverage         60.16%   59.94%   -0.23%     
================================================
  Files               111      111              
  Lines              6123     6184      +61     
================================================
+ Hits               3684     3707      +23     
- Misses             2439     2477      +38     
Impacted Files Coverage Δ
python/cugraph/cugraph/structure/number_map.py 67.84% <ø> (ø)
...ugraph/cugraph/dask/structure/mg_property_graph.py 15.10% <7.50%> (-0.81%) :arrow_down:
python/cugraph/cugraph/dask/community/louvain.py 24.24% <27.27%> (ø)
python/cugraph/cugraph/community/louvain.py 100.00% <100.00%> (ø)
python/cugraph/cugraph/dask/comms/comms.py 34.06% <100.00%> (ø)
python/cugraph/cugraph/structure/property_graph.py 96.88% <100.00%> (-0.29%) :arrow_down:
python/cugraph/cugraph/testing/utils.py 79.14% <100.00%> (ø)
python/pylibcugraph/pylibcugraph/__init__.py 100.00% <100.00%> (ø)
...n/pylibcugraph/pylibcugraph/utilities/api_tools.py 71.87% <0.00%> (-6.25%) :arrow_down:
... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov-commenter avatar Sep 19 '22 06:09 codecov-commenter

rerun tests

BradReesWork avatar Sep 23 '22 17:09 BradReesWork

rerun tests

jnke2016 avatar Sep 26 '22 23:09 jnke2016

rerun tests

ChuckHastings avatar Sep 27 '22 17:09 ChuckHastings

rerun tests

BradReesWork avatar Sep 30 '22 20:09 BradReesWork

@gpucibot merge

BradReesWork avatar Sep 30 '22 20:09 BradReesWork