-
Notifications
You must be signed in to change notification settings - Fork 311
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
[REVIEW] enable multigraph #1280
Conversation
Please update the changelog in order to start CI tests. View the gpuCI docs here. |
Codecov Report
@@ Coverage Diff @@
## branch-0.17 #1280 +/- ##
===============================================
+ Coverage 60.38% 60.70% +0.32%
===============================================
Files 67 67
Lines 3029 3031 +2
===============================================
+ Hits 1829 1840 +11
+ Misses 1200 1191 -9
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.
I think it is best to capture our multigraph discussions in the form of multigraph class documentation and add it as part of this PR from the start while it is still fresh in our mind.
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, just a few minor requests.
No description provided.