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

[skip-ci] Cost Matrix first version #2377

Merged
merged 7 commits into from
Jun 30, 2022
Merged

[skip-ci] Cost Matrix first version #2377

merged 7 commits into from
Jun 30, 2022

Conversation

acostadon
Copy link
Contributor

Create an end to end application of creating a cost matrix.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@acostadon acostadon added feature request New feature or request demo non-breaking Non-breaking change labels Jun 28, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jun 28, 2022 •
edited
Loading

Codecov Report

Merging #2377 (b55921a) into branch-22.08 (fa701f5) will increase coverage by 0.01%.
The diff coverage is n/a.

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

@@               Coverage Diff                @@
##           branch-22.08    #2377      +/-   ##
================================================
+ Coverage         60.08%   60.10%   +0.01%     
================================================
  Files               102      102              
  Lines              5158     5158              
================================================
+ Hits               3099     3100       +1     
+ Misses             2059     2058       -1
Impacted Files Coverage Δ
...ython/cugraph/cugraph/community/ktruss_subgraph.py 88.23% <0.00%> (+2.94%) ⬆️

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 fa701f5...a4f03c6. Read the comment docs.

@BradReesWork BradReesWork added this to the 22.08 milestone Jun 28, 2022
@acostadon acostadon marked this pull request as ready for review June 29, 2022 14:35
@acostadon acostadon requested a review from a team as a code owner June 29, 2022 14:35
@acostadon acostadon self-assigned this Jun 29, 2022
@BradReesWork BradReesWork self-requested a review June 29, 2022 18:53
BradReesWork
BradReesWork requested changes Jun 29, 2022
View reviewed changes
Copy link
Member

@BradReesWork BradReesWork left a comment

Choose a reason for hiding this comment

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

Needs a skip notebook test

@BradReesWork
Copy link
Member

rerun tests

@BradReesWork BradReesWork changed the title Cost Matrix first version [skip-ci] Cost Matrix first version Jun 30, 2022
@BradReesWork
Copy link
Member

@gpucibot merge

@BradReesWork
Copy link
Member

rerun tests

@rapids-bot rapids-bot bot merged commit fe776f2 into rapidsai:branch-22.08 Jun 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BradReesWork BradReesWork BradReesWork approved these changes

Assignees

@acostadon acostadon

Labels
feature request New feature or request non-breaking Non-breaking change
Projects
None yet
Milestone
22.08
Development

Successfully merging this pull request may close these issues.
3 participants
@acostadon @codecov-commenter @BradReesWork