commit a9ebf2d03a39f4c72453bdf3e2eae7a27988b021 Author: Karsten Loesing <karsten.loesing@gmx.net> Date: Tue Oct 23 12:27:27 2012 -0400 Stop after removing last but one relay (#1854). Calculating the maximum probability difference between pristine and empty consensus doesn't make much sense. More importantly, the current implementation violates the function relationship between linf and min_adv_bw, because we're not increasing min_adv_bw in the last run and thereby outputting two different linfs for the same min_adv_bw. Should be safe to omit that final run, because we're not looking for the case where we remove all relays from the consensus anyway. I hope. --- task-1854/pylinf.py | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/task-1854/pylinf.py b/task-1854/pylinf.py index 3bc6de1..1a6e2d2 100644 --- a/task-1854/pylinf.py +++ b/task-1854/pylinf.py @@ -121,7 +121,7 @@ def run(file_name): omitted_routers = 0 min_adv_bw = routers[0].advertised_bw - while(omitted_routers<=len(routers)): + while(omitted_routers<len(routers)): total_bw = 0 # this is the difference btw probability of choosing a relay in pristine