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

Bug louvain reverted fix #1766

Merged

Conversation

ChuckHastings
Copy link
Collaborator

One of the recent PRs to migrate things from the experimental namespace into the cugraph namespace accidentally overwrote this bug fix. Reapplying it.

@ChuckHastings ChuckHastings requested a review from a team as a code owner August 6, 2021 23:20
@ChuckHastings ChuckHastings self-assigned this Aug 6, 2021
@ChuckHastings ChuckHastings added 3 - Ready for Review bug Something isn't working non-breaking Non-breaking change labels Aug 6, 2021
@ChuckHastings ChuckHastings added this to the 21.10 milestone Aug 6, 2021
@ChuckHastings ChuckHastings added this to Algorithms and Primitives in 21.10 Release Aug 6, 2021
@codecov-commenter
Copy link

codecov-commenter commented Aug 7, 2021

Codecov Report

❗ No coverage uploaded for pull request base (branch-21.10@70b4ad3). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head bd7f683 differs from pull request most recent head a967638. Consider uploading reports for the commit a967638 to get more accurate results
Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.10    #1766   +/-   ##
===============================================
  Coverage                ?   59.75%           
===============================================
  Files                   ?       77           
  Lines                   ?     3523           
  Branches                ?        0           
===============================================
  Hits                    ?     2105           
  Misses                  ?     1418           
  Partials                ?        0           

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 70b4ad3...a967638. Read the comment docs.

@BradReesWork BradReesWork moved this from Algorithms and Primitives to PRs in 21.10 Release Aug 9, 2021
@BradReesWork
Copy link
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit bff09c0 into rapidsai:branch-21.10 Aug 10, 2021
21.10 Release automation moved this from PRs to Done Aug 10, 2021
@ChuckHastings ChuckHastings deleted the bug_louvain_reverted_fix branch February 1, 2022 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working non-breaking Non-breaking change
Projects
No open projects
21.10 Release
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants