-
Notifications
You must be signed in to change notification settings - Fork 182
Conversation
What do you have in mind? |
I have a few ideas but they all have flaws. Maybe we can make a combination of them
How about |
Codecov Report
@@ Coverage Diff @@
## master #1039 +/- ##
==========================================
+ Coverage 99.82% 99.82% +<.01%
==========================================
Files 86 89 +3
Lines 2821 2847 +26
==========================================
+ Hits 2816 2842 +26
Misses 5 5 |
What about calling it "Combinatorics" ? |
@sbromberger what do you think? |
I don't have a well-formed opinion on the naming, but if this PR is ready for review, I'm game :) I'll probably want @jpfairbanks or @simonschoelly to review as well. |
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.
What is DegreeDominatingSet
? I think this should be explained somewhere.
Do these algorithms give some approximation guarantee such as outcome <= 2 * OPT
? Then you could mention that in the documentation.
I agree with @simonschoelly's comments. |
7ff2625
to
498553f
Compare
@simonschoelly there aren't approximation factors for these algos that I'm aware of |
I'm ok with it being part of this PR, but it should probably go into utils.jl. |
570823a
to
8e0f9cc
Compare
@sbromberger I think Travis must be restarted. Please take a look |
I think this is ready |
Isn't your algorithm the one described here: http://ac.informatik.uni-freiburg.de/teaching/ss_12/netalg/lectures/chapter7.pdf ? Then it has an approximation factor. |
a74b71e
to
cf766a7
Compare
Yeah you're right, the approximation factor is |
Greedy heuristics for Dominating Set.
We really should consider changing the organisation of these heuristics for NP-Complete optimisation problems (
vertex_cover
,independent_set
,dominating_set
).cc: @somil55 @sbromberger