-
Notifications
You must be signed in to change notification settings - Fork 312
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] Update BFS and SSSP to check start/source vertex for validity #2268
[BUG] Update BFS and SSSP to check start/source vertex for validity #2268
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
…/github.com/alexbarghi-nv/cugraph into branch-22.06_validate_start_vertex_bfs_sssp
Codecov Report
@@ Coverage Diff @@
## branch-22.06 #2268 +/- ##
================================================
- Coverage 70.82% 69.75% -1.08%
================================================
Files 170 175 +5
Lines 11036 11495 +459
================================================
+ Hits 7816 8018 +202
- Misses 3220 3477 +257
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you for also adding the additional tests.
@gpucibot merge |
Adds a check in BFS and SSSP to ensure that the start vertex is a valid vertex in the provided graph. Returns a ValueError if the start vertex is invalid. Closes #1587