Skip to content

Fix triangle count#2325

Merged
rapids-bot[bot] merged 3 commits intorapidsai:branch-22.08from
ChuckHastings:fix_triangle_count
Jun 16, 2022
Merged

Fix triangle count#2325
rapids-bot[bot] merged 3 commits intorapidsai:branch-22.08from
ChuckHastings:fix_triangle_count

Conversation

@ChuckHastings
Copy link
Collaborator

Fixed MG issue discovered by @jnke2016.

Triangle count test was not correct for MG. If vertices are specified, the output size isn't known until after vertices are shuffled to the correct GPU.

@ChuckHastings ChuckHastings requested a review from a team as a code owner June 2, 2022 00:53
@ChuckHastings ChuckHastings self-assigned this Jun 2, 2022
@ChuckHastings ChuckHastings added bug Something isn't working 3 - Ready for Review non-breaking Non-breaking change labels Jun 2, 2022
@ChuckHastings ChuckHastings added this to the 22.08 milestone Jun 2, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jun 2, 2022

Codecov Report

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

@@               Coverage Diff               @@
##             branch-22.08    #2325   +/-   ##
===============================================
  Coverage                ?   61.45%           
===============================================
  Files                   ?      107           
  Lines                   ?     5759           
  Branches                ?        0           
===============================================
  Hits                    ?     3539           
  Misses                  ?     2220           
  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 4922450...bb85ebe. Read the comment docs.

@ChuckHastings
Copy link
Collaborator Author

rerun tests

cugraph_type_erased_device_array_view_t* p_start_view = NULL;

int rank = 0;
rank = cugraph_resource_handle_get_rank(handle);
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry for nitpicking, but int rank = cugraph_resource_handle_get_rank(handle); might be cleaner.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

@ChuckHastings
Copy link
Collaborator Author

rerun tests

5 similar comments
@ChuckHastings
Copy link
Collaborator Author

rerun tests

@ChuckHastings
Copy link
Collaborator Author

rerun tests

@ChuckHastings
Copy link
Collaborator Author

rerun tests

@ChuckHastings
Copy link
Collaborator Author

rerun tests

@ChuckHastings
Copy link
Collaborator Author

rerun tests

@ChuckHastings
Copy link
Collaborator Author

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 9578cfe into rapidsai:branch-22.08 Jun 16, 2022
@ChuckHastings ChuckHastings deleted the fix_triangle_count branch August 4, 2022 18:22
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

None yet

Development

Successfully merging this pull request may close these issues.

3 participants