-
Notifications
You must be signed in to change notification settings - Fork 504
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
fix ring assert error and handle two A->B not ring #939
Conversation
Change-Id: Iaff8c4f16e34eb12f0fed2b33487735f783ae0ec
Codecov Report
@@ Coverage Diff @@
## master #939 +/- ##
============================================
+ Coverage 71.73% 71.95% +0.22%
- Complexity 4648 4674 +26
============================================
Files 298 298
Lines 22317 22317
Branches 3157 3157
============================================
+ Hits 16009 16059 +50
+ Misses 4741 4697 -44
+ Partials 1567 1561 -6 Continue to review full report at Codecov.
|
} else { | ||
hasInEdge = true; | ||
} | ||
if (hasOutEdge && hasInEdge) { |
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.
still returned 3 vertices
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.
oh seems the graph itself exist both direction edges between tow vertices
} else { | ||
hasInEdge = true; | ||
} | ||
if (hasOutEdge && hasInEdge) { |
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.
oh seems the graph itself exist both direction edges between tow vertices
Change-Id: Iaff8c4f16e34eb12f0fed2b33487735f783ae0ec