-
Notifications
You must be signed in to change notification settings - Fork 120
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
Graph Equivalence Changes #491
Conversation
emmanvg
commented
Feb 16, 2021
- fix in algorithm to guarantee symmetry
- add different functions for "similarity" and "equivalence"
busted main loop, symmetrical properties not present
- new methods for graph equivalence and similarity - remove sorting and len comparisons - rename some variables
Codecov Report
@@ Coverage Diff @@
## master #491 +/- ##
==========================================
- Coverage 89.27% 89.27% -0.01%
==========================================
Files 146 147 +1
Lines 16032 16442 +410
==========================================
+ Hits 14313 14678 +365
- Misses 1719 1764 +45
Continue to review full report at Codecov.
|
…ve redundant object_similarity call update test suite
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.
Looking good :) just a few ideas for small improvements.
Co-authored-by: Chris Lenk <[email protected]>
update variable names Co-authored-by: Chris Lenk <[email protected]>
Co-authored-by: Chris Lenk <[email protected]>
docstrings Co-authored-by: Chris Lenk <[email protected]>
docstrings Co-authored-by: Chris Lenk <[email protected]>
docstrings Co-authored-by: Chris Lenk <[email protected]>
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 great, thanks @emmanvg!