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.

Sign up for GitHub

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

Jump to bottom

add debug print for betweenness centrality, fix typo #2369

Merged
merged 8 commits into from
Jul 5, 2022
Merged

add debug print for betweenness centrality, fix typo #2369

merged 8 commits into from
Jul 5, 2022

Conversation

jnke2016
Copy link
Contributor

@jnke2016 jnke2016 commented Jun 22, 2022 •
edited
Loading

This PR

  1. adds/updates debug print for betweenness centrality
  2. remove legacy path for katz and eigenvector centrality

@jnke2016 jnke2016 requested a review from a team as a code owner June 22, 2022 17:37
@jnke2016 jnke2016 requested a review from a team as a code owner June 22, 2022 17:54
@ChuckHastings ChuckHastings added bug Something isn't working DO NOT MERGE Hold off on merging; see PR for details non-breaking Non-breaking change labels Jun 22, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jun 22, 2022 •
edited
Loading

Codecov Report

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

❗ Current head 11d7ef6 differs from pull request most recent head d397d07. Consider uploading reports for the commit d397d07 to get more accurate results

@@               Coverage Diff               @@
##             branch-22.08    #2369   +/-   ##
===============================================
  Coverage                ?   60.13%           
===============================================
  Files                   ?      102           
  Lines                   ?     5155           
  Branches                ?        0           
===============================================
  Hits                    ?     3100           
  Misses                  ?     2055           
  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 3a27fa5...d397d07. Read the comment docs.

@seunghwak seunghwak added 3 - Ready for Review and removed DO NOT MERGE Hold off on merging; see PR for details labels Jun 22, 2022
@ChuckHastings
Copy link
Collaborator

rerun tests

@BradReesWork BradReesWork added this to the 22.08 milestone Jun 29, 2022
@BradReesWork
Copy link
Member

rerun tests

@jnke2016 jnke2016 changed the title add debug print for betweenness centrality, fix typo add debug print for betweenness centrality, fix typo (DO NOT MERGE YET) Jun 29, 2022
@jnke2016 jnke2016 changed the title add debug print for betweenness centrality, fix typo (DO NOT MERGE YET) add debug print for betweenness centrality, fix typo Jul 5, 2022
@BradReesWork
Copy link
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 3385189 into rapidsai:branch-22.08 Jul 5, 2022
@jnke2016 jnke2016 deleted the branch-22.08_add_debug_print branch September 24, 2022 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChuckHastings ChuckHastings ChuckHastings approved these changes

@BradReesWork BradReesWork BradReesWork approved these changes

@rlratzel rlratzel rlratzel approved these changes

Assignees
No one assigned
Labels
bug Something isn't working non-breaking Non-breaking change
Projects
None yet
Milestone
22.08
Development

Successfully merging this pull request may close these issues.
6 participants
@jnke2016 @codecov-commenter @ChuckHastings @BradReesWork @rlratzel @seunghwak