-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
changed edge weights for max weight matching and added test #6145
changed edge weights for max weight matching and added test #6145
Conversation
c5b0041
to
db66b18
Compare
db66b18
to
798d30a
Compare
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.
Thanks for splitting this out @rfulekjames , this LGTM; I had just one question about the wording for the newly added docstring summary. I also took the liberty of pushing up a minor formatting fix just to correct some spacing/linebreaking for the new sentence - be sure to pull that change down!
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.
LGTM, thanks @rfulekjames !
changed edge weights for max weight matching and added test. Co-authored-by: Radoslav Fulek <rado@comitycard.com> Co-authored-by: Ross Barnowski <rossbar@berkeley.edu>
changed edge weights for max weight matching and added test. Co-authored-by: Radoslav Fulek <rado@comitycard.com> Co-authored-by: Ross Barnowski <rossbar@berkeley.edu>
changed edge weights for max weight matching and added test. Co-authored-by: Radoslav Fulek <rado@comitycard.com> Co-authored-by: Ross Barnowski <rossbar@berkeley.edu>
As discussed here we are splitting up the PR into two.
Fixes #5952
eulerize
function along the lines with the discussion of a small bug in eulerian.eulerize ? #5952 so that the minimum possible number of new edges is added in the resulting graph.